From: Sabrina Dubroca <s...@queasysnail.net> Date: Mon, 16 Nov 2015 22:54:20 +0100
> Reset pskb in macvlan_handle_frame in case skb_share_check returned a > clone. > > Fixes: 8a4eb5734e8d ("net: introduce rx_handler results and logic around > that") > Signed-off-by: Sabrina Dubroca <s...@queasysnail.net> Thanks for finding all of these bugs which seem to have the same signature. I always find pass by reference handling of SKBs to be extremely error prone, and it's a very inefficient calling convention anyways. I wondered if we could do better with the interface for rx_handler, for example returning an error-pointer for the 'new_skb', but nothing I came up with could provide the same semantics we have now. Sigh... -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html