diff options
author | Andrew Lutomirski <luto@amacapital.net> | 2014-04-17 00:41:34 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-05-31 00:52:15 -0400 |
commit | eac664d2832178c1d3a5e7560a26c10001717b77 (patch) | |
tree | 55cd18545fc8724151d52779cec90822fcbe63b4 /net/packet/diag.c | |
parent | e5eae4a0511241959498b180fa0df0d4f1b11b9c (diff) |
net: Fix ns_capable check in sock_diag_put_filterinfo
[ Upstream commit 78541c1dc60b65ecfce5a6a096fc260219d6784e ]
The caller needs capabilities on the namespace being queried, not on
their own namespace. This is a security bug, although it likely has
only a minor impact.
Cc: stable@vger.kernel.org
Signed-off-by: Andy Lutomirski <luto@amacapital.net>
Acked-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'net/packet/diag.c')
-rw-r--r-- | net/packet/diag.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/packet/diag.c b/net/packet/diag.c index a9584a2f6d69..ec8b6e8a80b1 100644 --- a/net/packet/diag.c +++ b/net/packet/diag.c | |||
@@ -171,7 +171,7 @@ static int sk_diag_fill(struct sock *sk, struct sk_buff *skb, | |||
171 | goto out_nlmsg_trim; | 171 | goto out_nlmsg_trim; |
172 | 172 | ||
173 | if ((req->pdiag_show & PACKET_SHOW_FILTER) && | 173 | if ((req->pdiag_show & PACKET_SHOW_FILTER) && |
174 | sock_diag_put_filterinfo(user_ns, sk, skb, PACKET_DIAG_FILTER)) | 174 | sock_diag_put_filterinfo(sk, skb, PACKET_DIAG_FILTER)) |
175 | goto out_nlmsg_trim; | 175 | goto out_nlmsg_trim; |
176 | 176 | ||
177 | return nlmsg_end(skb, nlh); | 177 | return nlmsg_end(skb, nlh); |