lkml.org 
[lkml]   [2021]   [Apr]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.14 01/52] selinux: vsock: Set SID for socket returned by accept()
    Date
    From: David Brazdil <dbrazdil@google.com>

    [ Upstream commit 1f935e8e72ec28dddb2dc0650b3b6626a293d94b ]

    For AF_VSOCK, accept() currently returns sockets that are unlabelled.
    Other socket families derive the child's SID from the SID of the parent
    and the SID of the incoming packet. This is typically done as the
    connected socket is placed in the queue that accept() removes from.

    Reuse the existing 'security_sk_clone' hook to copy the SID from the
    parent (server) socket to the child. There is no packet SID in this
    case.

    Fixes: d021c344051a ("VSOCK: Introduce VM Sockets")
    Signed-off-by: David Brazdil <dbrazdil@google.com>
    Signed-off-by: David S. Miller <davem@davemloft.net>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    net/vmw_vsock/af_vsock.c | 1 +
    1 file changed, 1 insertion(+)

    diff --git a/net/vmw_vsock/af_vsock.c b/net/vmw_vsock/af_vsock.c
    index eafcc75f289a..ae85a5e5648b 100644
    --- a/net/vmw_vsock/af_vsock.c
    +++ b/net/vmw_vsock/af_vsock.c
    @@ -635,6 +635,7 @@ struct sock *__vsock_create(struct net *net,
    vsk->trusted = psk->trusted;
    vsk->owner = get_cred(psk->owner);
    vsk->connect_timeout = psk->connect_timeout;
    + security_sk_clone(parent, sk);
    } else {
    vsk->trusted = ns_capable_noaudit(&init_user_ns, CAP_NET_ADMIN);
    vsk->owner = get_current_cred();
    --
    2.30.1


    \
     
     \ /
      Last update: 2021-04-05 11:10    [W:6.394 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site