diff options
author | Simon Wunderlich <siwu@hrz.tu-chemnitz.de> | 2010-05-07 15:47:26 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-05-11 16:42:40 -0400 |
commit | 7d02d777d4a7eb551999a35f52480c9ddac3d874 (patch) | |
tree | edb74b5bf7e9570deb26288af27885a61943e20e /drivers/staging/batman-adv | |
parent | 19dae340d2af3074abad5b4c7306ae240068f89f (diff) |
Staging: batman-adv: kfree_skb() in interface_tx() in error case
As we always return that the we consumed the skb, we should also free the skb
in the case of an error.
Signed-off-by: Simon Wunderlich <siwu@hrz.tu-chemnitz.de>
Signed-off-by: Andrew Lunn <andrew@lunn.ch>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/batman-adv')
-rw-r--r-- | drivers/staging/batman-adv/soft-interface.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/staging/batman-adv/soft-interface.c b/drivers/staging/batman-adv/soft-interface.c index 14b5ccaeaca..c3b52885b08 100644 --- a/drivers/staging/batman-adv/soft-interface.c +++ b/drivers/staging/batman-adv/soft-interface.c | |||
@@ -275,6 +275,7 @@ unlock: | |||
275 | spin_unlock_irqrestore(&orig_hash_lock, flags); | 275 | spin_unlock_irqrestore(&orig_hash_lock, flags); |
276 | dropped: | 276 | dropped: |
277 | priv->stats.tx_dropped++; | 277 | priv->stats.tx_dropped++; |
278 | kfree_skb(skb); | ||
278 | end: | 279 | end: |
279 | return NETDEV_TX_OK; | 280 | return NETDEV_TX_OK; |
280 | } | 281 | } |