Commit db83d136 authored Jan 09, 2012 by Manfred Rudigier Committed by David S. Miller Jan 12, 2012

gianfar: Fix missing sock reference when processing TX time stamps

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>

parent eb8a54a7

drivers/net/ethernet/freescale/gianfar.c

+4 −0

Original line number	Original line	Diff line number	Diff line
	@@ -2086,6 +2086,10 @@ static int gfar_start_xmit(struct sk_buff skb, struct net_device dev)
	kfree_skb(skb);		kfree_skb(skb);
	return NETDEV_TX_OK;		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);		kfree_skb(skb);
	skb = skb_new;		skb = skb_new;
	}		}