diff options
| author | Jakub Sitnicki <[email protected]> | 2020-02-06 11:16:51 +0000 |
|---|---|---|
| committer | Daniel Borkmann <[email protected]> | 2020-02-07 21:36:26 +0000 |
| commit | 0b2dc83906cf1e694e48003eae5df8fa63f76fd9 (patch) | |
| tree | 2cb216e87bfd171ea95e8af962f20ed0fa2a3e96 /tools/testing/selftests/bpf/prog_tests/sockmap_basic.c | |
| parent | bpf, sockmap: Don't sleep while holding RCU lock on tear-down (diff) | |
| download | kernel-0b2dc83906cf1e694e48003eae5df8fa63f76fd9.tar.gz kernel-0b2dc83906cf1e694e48003eae5df8fa63f76fd9.zip | |
bpf, sockhash: Synchronize_rcu before free'ing map
We need to have a synchronize_rcu before free'ing the sockhash because any
outstanding psock references will have a pointer to the map and when they
use it, this could trigger a use after free.
This is a sister fix for sockhash, following commit 2bb90e5cc90e ("bpf:
sockmap, synchronize_rcu before free'ing map") which addressed sockmap,
which comes from a manual audit.
Fixes: 604326b41a6fb ("bpf, sockmap: convert to generic sk_msg interface")
Signed-off-by: Jakub Sitnicki <[email protected]>
Signed-off-by: Daniel Borkmann <[email protected]>
Acked-by: John Fastabend <[email protected]>
Link: https://lore.kernel.org/bpf/[email protected]
Diffstat (limited to 'tools/testing/selftests/bpf/prog_tests/sockmap_basic.c')
0 files changed, 0 insertions, 0 deletions