Subject: RE: [PATCH bpf v2 1/4] bpf, sockmap: Don't let sock_map_{close,destroy,unhash} call itself

From: John Fastabend <[email-masked]>
Date: Tue, 24 Jan 2023 21:20:55 -0800

Jakub Sitnicki wrote:
> 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/[email-masked]/
> 
> Suggested-by: Eric Dumazet <[email-masked]>
> Signed-off-by: Jakub Sitnicki <[email-masked]>
> ---

LGTM and warn once logic should be fine IMO.

Acked-by: John Fastabend <[email-masked]>