diff options
| author | Richard Cochran <richardcochran@gmail.com> | 2011-10-21 02:49:16 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2011-10-25 01:14:50 +0200 |
| commit | f5ff7cd1a84caa9545d952a37ac872ccb73825fb (patch) | |
| tree | 13d027308cfd1e3557a7824f6af48eb761e390a1 | |
| parent | ipv6: Do not use routes from locally generated RAs (diff) | |
| download | linux-f5ff7cd1a84caa9545d952a37ac872ccb73825fb.tar.xz linux-f5ff7cd1a84caa9545d952a37ac872ccb73825fb.zip | |
dp83640: use proper function to free transmit time stamping packets
The previous commit enforces a new rule for handling the cloned packets
for transmit time stamping. These packets must not be freed using any other
function than skb_complete_tx_timestamp. This commit fixes the one and only
driver using this API.
The driver first appeared in v3.0.
Signed-off-by: Richard Cochran <richard.cochran@omicron.at>
Acked-by: Eric Dumazet <eric.dumazet@gmail.com>
Cc: <stable@vger.kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/phy/dp83640.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/phy/dp83640.c b/drivers/net/phy/dp83640.c index c588a162050f..13e571325a9c 100644 --- a/drivers/net/phy/dp83640.c +++ b/drivers/net/phy/dp83640.c @@ -1192,7 +1192,7 @@ static void dp83640_txtstamp(struct phy_device *phydev, case HWTSTAMP_TX_ONESTEP_SYNC: if (is_sync(skb, type)) { - kfree_skb(skb); + skb_complete_tx_timestamp(skb, NULL); return; } /* fall through */ @@ -1203,7 +1203,7 @@ static void dp83640_txtstamp(struct phy_device *phydev, case HWTSTAMP_TX_OFF: default: - kfree_skb(skb); + skb_complete_tx_timestamp(skb, NULL); break; } } |