diff options
| author |   John Fastabend <john.fastabend@gmail.com> | 2018-07-05 08:50:04 -0700 |
|---|---|---|
| committer |   Alexei Starovoitov <ast@kernel.org> | 2018-07-07 15:19:30 -0700 |
| commit | 99ba2b5aba24e022683a7db63204f9e306fe7ab9 (patch) | |
| tree | e174736f65a9dd001a67e2f007f93b5fd6265992 /net/core/filter.c | |
| parent | bpf: fix sk_skb programs without skb->dev assigned (diff) | |
| download | linux-dev-99ba2b5aba24e022683a7db63204f9e306fe7ab9.tar.xz linux-dev-99ba2b5aba24e022683a7db63204f9e306fe7ab9.zip | |
bpf: sockhash, disallow bpf_tcp_close and update in parallel
After latest lock updates there is no longer anything preventing a
close and recvmsg call running in parallel. Additionally, we can
race update with close if we close a socket and simultaneously update
if via the BPF userspace API (note the cgroup ops are already run
with sock_lock held).
To resolve this take sock_lock in close and update paths.
Reported-by: syzbot+b680e42077a0d7c9a0c4@syzkaller.appspotmail.com
Fixes: e9db4ef6bf4c ("bpf: sockhash fix omitted bucket lock in sock_close")
Signed-off-by: John Fastabend <john.fastabend@gmail.com>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Diffstat (limited to 'net/core/filter.c')
0 files changed, 0 insertions, 0 deletions