packet: Fix leak in pre-defrag support.
authorDavid S. Miller <davem@davemloft.net>
Wed, 6 Jul 2011 14:30:59 +0000 (07:30 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 6 Jul 2011 14:30:59 +0000 (07:30 -0700)
When we clone the SKB, we forget about the original
one.  Avoid this problem by using skb_share_check().

Reported-by: Penttilä Mika <mika.penttila@ixonos.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/packet/af_packet.c

index 41f0489ff665f8533ac0e05dd7f8c77e956e8375..aec50a1e9849ce74d1ff1bc520e998e5fc0e4674 100644 (file)
@@ -476,7 +476,7 @@ static struct sk_buff *fanout_check_defrag(struct sk_buff *skb)
                return skb;
 
        if (ip_is_fragment(ip_hdr(skb))) {
-               skb = skb_clone(skb, GFP_ATOMIC);
+               skb = skb_share_check(skb, GFP_ATOMIC);
                if (skb) {
                        if (pskb_trim_rcsum(skb, len))
                                return skb;