diff options
author | Eric Dumazet <eric.dumazet@gmail.com> | 2009-11-18 20:28:10 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-11-19 16:16:21 -0500 |
commit | 56cf54831fd1be105e89a9df899e3b22442da180 (patch) | |
tree | 3e1ec732a0e26b5eba977750fdd50a52f1dc1fb4 | |
parent | 66b00a7c93ec782d118d2c03bd599cfd041e80a1 (diff) |
ieee802154: dont leak skbs in ieee802154_fake_xmit()
ieee802154_fake_xmit() should free skbs since it returns NETDEV_TX_OK
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Acked-by: Dmitry Eremin-Solenikov <dbaryshkov@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/ieee802154/fakehard.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/ieee802154/fakehard.c b/drivers/ieee802154/fakehard.c index 96a2959ce877..7c544f7c74c4 100644 --- a/drivers/ieee802154/fakehard.c +++ b/drivers/ieee802154/fakehard.c | |||
@@ -260,15 +260,12 @@ static int ieee802154_fake_close(struct net_device *dev) | |||
260 | static netdev_tx_t ieee802154_fake_xmit(struct sk_buff *skb, | 260 | static netdev_tx_t ieee802154_fake_xmit(struct sk_buff *skb, |
261 | struct net_device *dev) | 261 | struct net_device *dev) |
262 | { | 262 | { |
263 | skb->iif = dev->ifindex; | ||
264 | skb->dev = dev; | ||
265 | dev->stats.tx_packets++; | 263 | dev->stats.tx_packets++; |
266 | dev->stats.tx_bytes += skb->len; | 264 | dev->stats.tx_bytes += skb->len; |
267 | 265 | ||
268 | dev->trans_start = jiffies; | ||
269 | |||
270 | /* FIXME: do hardware work here ... */ | 266 | /* FIXME: do hardware work here ... */ |
271 | 267 | ||
268 | dev_kfree_skb(skb); | ||
272 | return NETDEV_TX_OK; | 269 | return NETDEV_TX_OK; |
273 | } | 270 | } |
274 | 271 | ||