net: Restrict unwhitelisted proto caches to size 0

kees · kees · commit 289a4860d1f5 · 2018-01-15T12:08:02.000-08:00
Now that protocols have been annotated (the copy of icsk_ca_ops-&gt;name
is of an ops field from outside the slab cache):

$ git grep 'copy_.*_user.*sk.*-&gt;'
caif/caif_socket.c: copy_from_user(&amp;cf_sk-&gt;conn_req.param.data, ov, ol)) {
ipv4/raw.c:   if (copy_from_user(&amp;raw_sk(sk)-&gt;filter, optval, optlen))
ipv4/raw.c:       copy_to_user(optval, &amp;raw_sk(sk)-&gt;filter, len))
ipv4/tcp.c:       if (copy_to_user(optval, icsk-&gt;icsk_ca_ops-&gt;name, len))
ipv4/tcp.c:       if (copy_to_user(optval, icsk-&gt;icsk_ulp_ops-&gt;name, len))
ipv6/raw.c:       if (copy_from_user(&amp;raw6_sk(sk)-&gt;filter, optval, optlen))
ipv6/raw.c:           if (copy_to_user(optval, &amp;raw6_sk(sk)-&gt;filter, len))
sctp/socket.c: if (copy_from_user(&amp;sctp_sk(sk)-&gt;subscribe, optval, optlen))
sctp/socket.c: if (copy_to_user(optval, &amp;sctp_sk(sk)-&gt;subscribe, len))
sctp/socket.c: if (copy_to_user(optval, &amp;sctp_sk(sk)-&gt;initmsg, len))

we can switch the default proto usercopy region to size 0. Any protocols
needing to add whitelisted regions must annotate the fields with the
useroffset and usersize fields of struct proto.

This patch is modified from Brad Spengler/PaX Team's PAX_USERCOPY
whitelisting code in the last public patch of grsecurity/PaX based on my
understanding of the code. Changes or omissions from the original code are
mine and don't reflect the original grsecurity/PaX code.

Cc: "David S. Miller" &lt;davem@davemloft.net&gt;
Cc: Eric Dumazet &lt;edumazet@google.com&gt;
Cc: Paolo Abeni &lt;pabeni@redhat.com&gt;
Cc: David Howells &lt;dhowells@redhat.com&gt;
Cc: netdev@vger.kernel.org
Signed-off-by: Kees Cook &lt;keescook@chromium.org&gt;
diff --git a/net/core/sock.c b/net/core/sock.c
@@ -3154,9 +3154,7 @@ int proto_register(struct proto *prot, int alloc_slab)
 		prot->slab = kmem_cache_create_usercopy(prot->name,
 					prot->obj_size, 0,
 					SLAB_HWCACHE_ALIGN | prot->slab_flags,
-					prot->usersize ? prot->useroffset : 0,
-					prot->usersize ? prot->usersize
-						       : prot->obj_size,
+					prot->useroffset, prot->usersize,
 					NULL);
 
 		if (prot->slab == NULL) {
