Thread (8 messages) flat view 8 messages, 3 authors, 2021-07-12

Re: [PATCH bpf v3 1/2] bpf, sockmap: fix potential memory leak on unlikely error case

From: John Fastabend <john.fastabend@gmail.com>
Date: 2021-07-08 20:39:18
Also in: bpf

Cong Wang wrote:
On Tue, Jul 6, 2021 at 9:31 AM John Fastabend [off-list ref] wrote:
quoted
If skb_linearize is needed and fails we could leak a msg on the error
handling. To fix ensure we kfree the msg block before returning error.
Found during code review.
sk_psock_skb_ingress_self() also needs the same fix, right?
Yep.
Other than this, it looks good to me.
I'll do another spin to get the other one as well. Mind as well
fix both cases at once.
Thanks.
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help