diff options
| author | Andrei Vagin <avagin@openvz.org> | 2017-10-25 10:16:42 -0700 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-11-18 11:11:06 +0100 |
| commit | 93b12f202a4eae6856e2f67e002c8e4363600d4a (patch) | |
| tree | 5c592c3db81dc565a6d3fffa948c7a239c35307a | |
| parent | 84237c54b2399076c7a22b7818b0a414099d02d0 (diff) | |
| download | linux-stable-93b12f202a4eae6856e2f67e002c8e4363600d4a.tar.gz linux-stable-93b12f202a4eae6856e2f67e002c8e4363600d4a.tar.bz2 linux-stable-93b12f202a4eae6856e2f67e002c8e4363600d4a.zip | |
net/unix: don't show information about sockets from other namespaces
[ Upstream commit 0f5da659d8f1810f44de14acf2c80cd6499623a0 ]
socket_diag shows information only about sockets from a namespace where
a diag socket lives.
But if we request information about one unix socket, the kernel don't
check that its netns is matched with a diag socket namespace, so any
user can get information about any unix socket in a system. This looks
like a bug.
v2: add a Fixes tag
Fixes: 51d7cccf0723 ("net: make sock diag per-namespace")
Signed-off-by: Andrei Vagin <avagin@openvz.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | net/unix/diag.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/unix/diag.c b/net/unix/diag.c index 4d9679701a6d..384c84e83462 100644 --- a/net/unix/diag.c +++ b/net/unix/diag.c @@ -257,6 +257,8 @@ static int unix_diag_get_exact(struct sk_buff *in_skb, err = -ENOENT; if (sk == NULL) goto out_nosk; + if (!net_eq(sock_net(sk), net)) + goto out; err = sock_diag_check_cookie(sk, req->udiag_cookie); if (err) |