Skip to content

Commit 034294f

Browse files
sm1ling-knightpcmoore
authored andcommitted
selinux: Fix SCTP error inconsistency in selinux_socket_bind()
Check sk->sk_protocol instead of security class to recognize SCTP socket. SCTP socket is initialized with SECCLASS_SOCKET class if policy does not support EXTSOCKCLASS capability. In this case bind(2) hook wrongfully return EAFNOSUPPORT instead of EINVAL. The inconsistency was detected with help of Landlock tests: https://lore.kernel.org/all/b58680ca-81b2-7222-7287-0ac7f4227c3c@huawei-partners.com/ Fixes: 0f8db8c ("selinux: add AF_UNSPEC and INADDR_ANY checks to selinux_socket_bind()") Signed-off-by: Mikhail Ivanov <ivanov.mikhail1@huawei-partners.com> Signed-off-by: Paul Moore <paul@paul-moore.com>
1 parent c75c794 commit 034294f

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

security/selinux/hooks.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4835,7 +4835,7 @@ static int selinux_socket_bind(struct socket *sock, struct sockaddr *address, in
48354835
return err;
48364836
err_af:
48374837
/* Note that SCTP services expect -EINVAL, others -EAFNOSUPPORT. */
4838-
if (sksec->sclass == SECCLASS_SCTP_SOCKET)
4838+
if (sk->sk_protocol == IPPROTO_SCTP)
48394839
return -EINVAL;
48404840
return -EAFNOSUPPORT;
48414841
}

0 commit comments

Comments
 (0)