security: add hook for socketpair()

Right now the LSM labels for socketpairs are always uninitialized,
since there is no security hook for the socketpair() syscall. This
patch adds the required hooks so LSMs can properly label socketpairs.
This allows SO_PEERSEC to return useful information on those sockets.

Note that the behavior of socketpair() can be emulated by creating a
listener socket, connecting to it, and then discarding the initial
listener socket. With this workaround, SO_PEERSEC would return the
caller's security context. However, with socketpair(), the uninitialized
context is returned unconditionally. This is unexpected and makes
socketpair() less useful in situations where the security context is
crucial to the application.

With the new socketpair-hook this disparity can be solved by making
socketpair() return the expected security context.

Acked-by: Serge Hallyn <serge@hallyn.com>
Signed-off-by: Tom Gundersen <teg@jklm.no>
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
Signed-off-by: James Morris <james.morris@microsoft.com>

diff --git a/include/linux/lsm_hooks.h b/include/linux/lsm_hooks.h
index 9d0b286f3dbaf56c5362327a1bc82ed1a9132a34..8f1131c8dd54b1a6f5586a793eef839036004cfa 100644 (file)
--- a/include/linux/lsm_hooks.h
+++ b/include/linux/lsm_hooks.h
@@ -757,6 +757,11 @@
  *     @type contains the requested communications type.
  *     @protocol contains the requested protocol.
  *     @kern set to 1 if a kernel socket.
+ * @socket_socketpair:
+ *     Check permissions before creating a fresh pair of sockets.
+ *     @socka contains the first socket structure.
+ *     @sockb contains the second socket structure.
+ *     Return 0 if permission is granted and the connection was established.
  * @socket_bind:
  *     Check permission before socket protocol layer bind operation is
  *     performed and the socket @sock is bound to the address specified in the
@@ -1656,6 +1661,7 @@ union security_list_options {
        int (*socket_create)(int family, int type, int protocol, int kern);
        int (*socket_post_create)(struct socket *sock, int family, int type,
                                        int protocol, int kern);
+       int (*socket_socketpair)(struct socket *socka, struct socket *sockb);
        int (*socket_bind)(struct socket *sock, struct sockaddr *address,
                                int addrlen);
        int (*socket_connect)(struct socket *sock, struct sockaddr *address,
@@ -1922,6 +1928,7 @@ struct security_hook_heads {
        struct hlist_head unix_may_send;
        struct hlist_head socket_create;
        struct hlist_head socket_post_create;
+       struct hlist_head socket_socketpair;
        struct hlist_head socket_bind;
        struct hlist_head socket_connect;
        struct hlist_head socket_listen;

diff --git a/include/linux/security.h b/include/linux/security.h
index ecb06e1357dd12bbcfd96eba526d673d344b8006..63030c85ee1927ae0de30a07148fa511e19231aa 100644 (file)
--- a/include/linux/security.h
+++ b/include/linux/security.h
@@ -1177,6 +1177,7 @@ int security_unix_may_send(struct socket *sock,  struct socket *other);
 int security_socket_create(int family, int type, int protocol, int kern);
 int security_socket_post_create(struct socket *sock, int family,
                                int type, int protocol, int kern);
+int security_socket_socketpair(struct socket *socka, struct socket *sockb);
 int security_socket_bind(struct socket *sock, struct sockaddr *address, int addrlen);
 int security_socket_connect(struct socket *sock, struct sockaddr *address, int addrlen);
 int security_socket_listen(struct socket *sock, int backlog);
@@ -1248,6 +1249,12 @@ static inline int security_socket_post_create(struct socket *sock,
        return 0;
 }
 
+static inline int security_socket_socketpair(struct socket *socka,
+                                            struct socket *sockb)
+{
+       return 0;
+}
+
 static inline int security_socket_bind(struct socket *sock,
                                       struct sockaddr *address,
                                       int addrlen)

diff --git a/security/security.c b/security/security.c
index 7bc2fde023a797e7a941622a0a6d45c11fdb8b6b..68f46d849abea30d148dac3942d833c8ef80e114 100644 (file)
--- a/security/security.c
+++ b/security/security.c
@@ -1358,6 +1358,12 @@ int security_socket_post_create(struct socket *sock, int family,
                                                protocol, kern);
 }
 
+int security_socket_socketpair(struct socket *socka, struct socket *sockb)
+{
+       return call_int_hook(socket_socketpair, 0, socka, sockb);
+}
+EXPORT_SYMBOL(security_socket_socketpair);
+
 int security_socket_bind(struct socket *sock, struct sockaddr *address, int addrlen)
 {
        return call_int_hook(socket_bind, 0, sock, address, addrlen);