Commit 0b2dc839 authored by Jakub Sitnicki's avatar Jakub Sitnicki Committed by Daniel Borkmann
Browse files

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 2bb90e5c ("bpf:
sockmap, synchronize_rcu before free'ing map") which addressed sockmap,
which comes from a manual audit.

Fixes: 604326b4

 ("bpf, sockmap: convert to generic sk_msg interface")
Signed-off-by: default avatarJakub Sitnicki <>
Signed-off-by: default avatarDaniel Borkmann <>
Acked-by: default avatarJohn Fastabend <>
parent db6a5018
......@@ -250,6 +250,7 @@ static void sock_map_free(struct bpf_map *map)
/* wait for psock readers accessing its map link */
......@@ -877,6 +878,9 @@ static void sock_hash_free(struct bpf_map *map)
/* wait for psock readers accessing its map link */
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment