diff options
| author |   John Fastabend <john.fastabend@gmail.com> | 2017-10-18 07:10:15 -0700 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2017-10-20 13:01:29 +0100 |
| commit | 435bf0d3f99a164df7e8c30428cef266b91d1d3b (patch) | |
| tree | a03f823017e2f876a308a8a8a782a14d580b3a82 /kernel | |
| parent | sctp: add the missing sock_owned_by_user check in sctp_icmp_redirect (diff) | |
| download | linux-dev-435bf0d3f99a164df7e8c30428cef266b91d1d3b.tar.xz linux-dev-435bf0d3f99a164df7e8c30428cef266b91d1d3b.zip | |
bpf: enforce TCP only support for sockmap
Only TCP sockets have been tested and at the moment the state change
callback only handles TCP sockets. This adds a check to ensure that
sockets actually being added are TCP sockets.
For net-next we can consider UDP support.
Signed-off-by: John Fastabend <john.fastabend@gmail.com>
Acked-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Alexei Starovoitov <ast@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/bpf/sockmap.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/kernel/bpf/sockmap.c b/kernel/bpf/sockmap.c index 6424ce0e4969..c68899d5b246 100644 --- a/kernel/bpf/sockmap.c +++ b/kernel/bpf/sockmap.c @@ -840,6 +840,12 @@ static int sock_map_update_elem(struct bpf_map *map, return -EINVAL; } + if (skops.sk->sk_type != SOCK_STREAM || + skops.sk->sk_protocol != IPPROTO_TCP) { + fput(socket->file); + return -EOPNOTSUPP; + } + err = sock_map_ctx_update_elem(&skops, map, key, flags); fput(socket->file); return err; |