OSDN Git Service

gianfar: Fix missing sock reference when processing TX time stamps
authorManfred Rudigier <Manfred.Rudigier@omicron.at>
Mon, 9 Jan 2012 23:26:50 +0000 (23:26 +0000)
committerDavid S. Miller <davem@davemloft.net>
Thu, 12 Jan 2012 23:26:01 +0000 (15:26 -0800)
When there is not enough headroom in the skb a private copy will be made.
However, the private copy had no reference to the socket and consequently
no time stamp could be queued on the socket error queue during the
skb_tstamp_tx function. This patch fixes this issue by also stealing the
sock reference from the original skb after making the private copy.

Signed-off-by: Manfred Rudigier <manfred.rudigier@omicron.at>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/freescale/gianfar.c

index e01cdaa..2b3c129 100644 (file)
@@ -2086,6 +2086,10 @@ static int gfar_start_xmit(struct sk_buff *skb, struct net_device *dev)
                        kfree_skb(skb);
                        return NETDEV_TX_OK;
                }
+
+               /* Steal sock reference for processing TX time stamps */
+               swap(skb_new->sk, skb->sk);
+               swap(skb_new->destructor, skb->destructor);
                kfree_skb(skb);
                skb = skb_new;
        }