BACKPORT: Smack: Assign smack_known_web as default smk_in label for kernel thread...
authorMarcin Lis <m.lis@samsung.com>
Thu, 22 Jan 2015 14:40:33 +0000 (15:40 +0100)
committerRafal Krypa <r.krypa@samsung.com>
Thu, 3 Nov 2016 16:41:55 +0000 (17:41 +0100)
This change fixes the bug associated with sockets owned by kernel threads. These
sockets, created usually by network devices' drivers tasks, received smk_in
label from the task that created them - the "floor" label in the most cases. The
result was that they were not able to receive data packets because of missing
smack rules. The main reason of the access deny is that the socket smk_in label
is placed as the object during smk check, kernel thread's capabilities are
omitted.

Signed-off-by: Marcin Lis <m.lis@samsung.com>
(cherry-picked from upstream 7412301b76bd53ee53b860f611fc3b5b1c2245b5)

security/smack/smack_lsm.c

index 9200322ceba6be99a835aab75c713e398687df12..6e7f15eaa1fee47e198634619b5147f5a194dd9e 100644 (file)
@@ -2453,7 +2453,21 @@ static int smack_inode_setsecurity(struct inode *inode, const char *name,
 static int smack_socket_post_create(struct socket *sock, int family,
                                    int type, int protocol, int kern)
 {
-       if (family != PF_INET || sock->sk == NULL)
+       struct socket_smack *ssp;
+
+       if (sock->sk == NULL)
+               return 0;
+
+       /*
+        * Sockets created by kernel threads receive web label.
+        */
+       if (unlikely(current->flags & PF_KTHREAD)) {
+               ssp = sock->sk->sk_security;
+               ssp->smk_in = &smack_known_web;
+               ssp->smk_out = &smack_known_web;
+       }
+
+       if (family != PF_INET)
                return 0;
        /*
         * Set the outbound netlbl.