diff options
| author | Simon Wunderlich <siwu@hrz.tu-chemnitz.de> | 2010-05-07 21:47:26 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-05-11 13:42:40 -0700 |
| commit | 7d02d777d4a7eb551999a35f52480c9ddac3d874 (patch) | |
| tree | edb74b5bf7e9570deb26288af27885a61943e20e | |
| 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>
| -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 14b5ccaeaca0..c3b52885b086 100644 --- a/drivers/staging/batman-adv/soft-interface.c +++ b/drivers/staging/batman-adv/soft-interface.c @@ -275,6 +275,7 @@ unlock: spin_unlock_irqrestore(&orig_hash_lock, flags); dropped: priv->stats.tx_dropped++; + kfree_skb(skb); end: return NETDEV_TX_OK; } |