selinux: fix the default socket labeling in sock_graft()
authorPaul Moore <pmoore@redhat.com>
Thu, 10 Jul 2014 14:17:48 +0000 (10:17 -0400)
committerPaul Moore <pmoore@redhat.com>
Thu, 10 Jul 2014 14:17:48 +0000 (10:17 -0400)
commit4da6daf4d3df5a977e4623963f141a627fd2efce
tree4dce3efc23da4d26f263b3f135f3edf2ffcf7798
parent170b5910d9fbea79de1bb40df22eda5f98250c0c
selinux: fix the default socket labeling in sock_graft()

The sock_graft() hook has special handling for AF_INET, AF_INET, and
AF_UNIX sockets as those address families have special hooks which
label the sock before it is attached its associated socket.
Unfortunately, the sock_graft() hook was missing a default approach
to labeling sockets which meant that any other address family which
made use of connections or the accept() syscall would find the
returned socket to be in an "unlabeled" state.  This was recently
demonstrated by the kcrypto/AF_ALG subsystem and the newly released
cryptsetup package (cryptsetup v1.6.5 and later).

This patch preserves the special handling in selinux_sock_graft(),
but adds a default behavior - setting the sock's label equal to the
associated socket - which resolves the problem with AF_ALG and
presumably any other address family which makes use of accept().

Cc: stable@vger.kernel.org
Signed-off-by: Paul Moore <pmoore@redhat.com>
Tested-by: Milan Broz <gmazyland@gmail.com>
include/linux/security.h
security/selinux/hooks.c