Files
linux/net/core
John Fastabend 733dcb4149 bpf, sockmap: Remove skb_orphan and let normal skb_kfree do cleanup
[ Upstream commit 10d58d0063 ]

Calling skb_orphan() is unnecessary in the strp rcv handler because the skb
is from a skb_clone() in __strp_recv. So it never has a destructor or a
sk assigned. Plus its confusing to read because it might hint to the reader
that the skb could have an sk assigned which is not true. Even if we did
have an sk assigned it would be cleaner to simply wait for the upcoming
kfree_skb().

Additionally, move the comment about strparser clone up so its closer to
the logic it is describing and add to it so that it is more complete.

Fixes: 604326b41a ("bpf, sockmap: convert to generic sk_msg interface")
Signed-off-by: John Fastabend <john.fastabend@gmail.com>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Link: https://lore.kernel.org/bpf/160226865548.5692.9098315689984599579.stgit@john-Precision-5820-Tower
Signed-off-by: Sasha Levin <sashal@kernel.org>
2020-10-29 10:07:49 +01:00
..
2020-05-15 17:29:41 +02:00
2020-09-26 18:05:23 +02:00
2020-04-20 12:43:24 -07:00
2020-03-29 22:30:57 -07:00
2019-12-12 17:07:05 -08:00
2020-09-12 14:22:14 +02:00
2019-07-22 20:47:56 -07:00