diff options
author | Jakub Sitnicki | 2023-01-21 13:41:43 +0100 |
---|---|---|
committer | Greg Kroah-Hartman | 2023-02-22 12:59:42 +0100 |
commit | 7499859881488da97589f3c79cc66fa75748ad49 (patch) | |
tree | 7883473a9730360e7584d1945d06649753ad497a /ipc/msg.c | |
parent | 22fcbb7802b98a121e9d7d3d8e42b498f070d6aa (diff) |
bpf, sockmap: Don't let sock_map_{close,destroy,unhash} call itself
[ Upstream commit 5b4a79ba65a1ab479903fff2e604865d229b70a9 ]
sock_map proto callbacks should never call themselves by design. Protect
against bugs like [1] and break out of the recursive loop to avoid a stack
overflow in favor of a resource leak.
[1] https://lore.kernel.org/all/00000000000073b14905ef2e7401@google.com/
Suggested-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: Jakub Sitnicki <jakub@cloudflare.com>
Acked-by: John Fastabend <john.fastabend@gmail.com>
Link: https://lore.kernel.org/r/20230113-sockmap-fix-v2-1-1e0ee7ac2f90@cloudflare.com
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'ipc/msg.c')
0 files changed, 0 insertions, 0 deletions