# Re: \[Patch bpf\] udp: fix a memory leak in udp\_read\_sock()

- From: Song Liu \<song@kernel.org\>
- Date: 2021-05-17 16:23:23 
- List: netdev
- Page: https://ratatoskr.run/netdev/2021/05/7768826
- Thread: https://ratatoskr.run/netdev/2021/05/7768819/t.md

```text
On Sun, May 16, 2021 at 8:33 PM Cong Wang [ref] wrote:
>
> From: Cong Wang <redacted>
>
> sk_psock_verdict_recv() clones the skb and uses the clone
> afterward, so udp_read_sock() should free the original skb after
> done using it.
>
> Fixes: d7f571188ecf ("udp: Implement ->read_sock() for sockmap")
> Cc: John Fastabend <john.fastabend@gmail.com>
> Cc: Daniel Borkmann <daniel@iogearbox.net>
> Cc: Jakub Sitnicki <jakub@cloudflare.com>
> Cc: Lorenz Bauer <redacted>
> Signed-off-by: Cong Wang <redacted>

Acked-by: Song Liu <song@kernel.org>
```
