net: Fix ns_capable check in sock_diag_put_filterinfo
authorAndrew Lutomirski <luto@amacapital.net>
Thu, 17 Apr 2014 04:41:34 +0000 (21:41 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 22 Apr 2014 16:49:39 +0000 (12:49 -0400)
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>
include/linux/sock_diag.h
net/core/sock_diag.c
net/packet/diag.c

index 54f91d35e5fd76f13b94d8297b10be6642e07887..302ab805b0bbba23ed7f4b8892b201e7e28469ce 100644 (file)
@@ -23,7 +23,7 @@ int sock_diag_check_cookie(void *sk, __u32 *cookie);
 void sock_diag_save_cookie(void *sk, __u32 *cookie);
 
 int sock_diag_put_meminfo(struct sock *sk, struct sk_buff *skb, int attr);
-int sock_diag_put_filterinfo(struct user_namespace *user_ns, struct sock *sk,
+int sock_diag_put_filterinfo(struct sock *sk,
                             struct sk_buff *skb, int attrtype);
 
 #endif
index d7af1885932269eb9f4196fe1211a6d09e298b97..9deb6abd6cf64da01fb5bcf8d85cf3baec21a618 100644 (file)
@@ -49,7 +49,7 @@ int sock_diag_put_meminfo(struct sock *sk, struct sk_buff *skb, int attrtype)
 }
 EXPORT_SYMBOL_GPL(sock_diag_put_meminfo);
 
-int sock_diag_put_filterinfo(struct user_namespace *user_ns, struct sock *sk,
+int sock_diag_put_filterinfo(struct sock *sk,
                             struct sk_buff *skb, int attrtype)
 {
        struct sock_fprog_kern *fprog;
@@ -58,7 +58,7 @@ int sock_diag_put_filterinfo(struct user_namespace *user_ns, struct sock *sk,
        unsigned int flen;
        int err = 0;
 
-       if (!ns_capable(user_ns, CAP_NET_ADMIN)) {
+       if (!ns_capable(sock_net(sk)->user_ns, CAP_NET_ADMIN)) {
                nla_reserve(skb, attrtype, 0);
                return 0;
        }
index 533ce4ff108ad94ff0a1e5205bc17f9c91c0b3ce..435ff99ba8c788d07b1682ce2b1a2fe7bd3eb0e6 100644 (file)
@@ -172,7 +172,7 @@ static int sk_diag_fill(struct sock *sk, struct sk_buff *skb,
                goto out_nlmsg_trim;
 
        if ((req->pdiag_show & PACKET_SHOW_FILTER) &&
-           sock_diag_put_filterinfo(user_ns, sk, skb, PACKET_DIAG_FILTER))
+           sock_diag_put_filterinfo(sk, skb, PACKET_DIAG_FILTER))
                goto out_nlmsg_trim;
 
        return nlmsg_end(skb, nlh);