[lkml]   [2018]   [Jan]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    Subject[PATCH 25/36] net: Restrict unwhitelisted proto caches to size 0
    Now that protocols have been annotated (the copy of icsk_ca_ops->name
    is of an ops field from outside the slab cache):

    $ git grep 'copy_.*_user.*sk.*->'
    caif/caif_socket.c: copy_from_user(&cf_sk->, ov, ol)) {
    ipv4/raw.c: if (copy_from_user(&raw_sk(sk)->filter, optval, optlen))
    ipv4/raw.c: copy_to_user(optval, &raw_sk(sk)->filter, len))
    ipv4/tcp.c: if (copy_to_user(optval, icsk->icsk_ca_ops->name, len))
    ipv4/tcp.c: if (copy_to_user(optval, icsk->icsk_ulp_ops->name, len))
    ipv6/raw.c: if (copy_from_user(&raw6_sk(sk)->filter, optval, optlen))
    ipv6/raw.c: if (copy_to_user(optval, &raw6_sk(sk)->filter, len))
    sctp/socket.c: if (copy_from_user(&sctp_sk(sk)->subscribe, optval, optlen))
    sctp/socket.c: if (copy_to_user(optval, &sctp_sk(sk)->subscribe, len))
    sctp/socket.c: if (copy_to_user(optval, &sctp_sk(sk)->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" <>
    Cc: Eric Dumazet <>
    Cc: Paolo Abeni <>
    Cc: David Howells <>
    Signed-off-by: Kees Cook <>
    net/core/sock.c | 4 +---
    1 file changed, 1 insertion(+), 3 deletions(-)

    diff --git a/net/core/sock.c b/net/core/sock.c
    index 261e6dbf0259..f39206b41b32 100644
    --- 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,

    if (prot->slab == NULL) {
     \ /
      Last update: 2018-01-14 23:20    [W:3.298 / U:0.020 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site