diff options
author | Jiri Slaby <jslaby@suse.cz> | 2010-10-10 19:26:56 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-10-11 14:13:59 -0400 |
commit | 419c20465d6319274e7286f0966e2390540e6e0a (patch) | |
tree | c4fca8f173fdd69b07a08455b34e0c01aacbe949 /drivers/net/pch_gbe | |
parent | 34d101dd6204bd100fc2e6f7b5f9a10f959ce2c9 (diff) |
NET: pch, fix use after free
Stanse found that pch_gbe_xmit_frame uses skb after it is freed. Fix
that.
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Cc: Masayuki Ohtake <masa-korg@dsn.okisemi.com>
Acked-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/pch_gbe')
-rw-r--r-- | drivers/net/pch_gbe/pch_gbe_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/pch_gbe/pch_gbe_main.c b/drivers/net/pch_gbe/pch_gbe_main.c index 53c56cf8aca2..e44644f169fd 100644 --- a/drivers/net/pch_gbe/pch_gbe_main.c +++ b/drivers/net/pch_gbe/pch_gbe_main.c | |||
@@ -1847,9 +1847,9 @@ static int pch_gbe_xmit_frame(struct sk_buff *skb, struct net_device *netdev) | |||
1847 | unsigned long flags; | 1847 | unsigned long flags; |
1848 | 1848 | ||
1849 | if (unlikely(skb->len > (adapter->hw.mac.max_frame_size - 4))) { | 1849 | if (unlikely(skb->len > (adapter->hw.mac.max_frame_size - 4))) { |
1850 | dev_kfree_skb_any(skb); | ||
1851 | pr_err("Transfer length Error: skb len: %d > max: %d\n", | 1850 | pr_err("Transfer length Error: skb len: %d > max: %d\n", |
1852 | skb->len, adapter->hw.mac.max_frame_size); | 1851 | skb->len, adapter->hw.mac.max_frame_size); |
1852 | dev_kfree_skb_any(skb); | ||
1853 | adapter->stats.tx_length_errors++; | 1853 | adapter->stats.tx_length_errors++; |
1854 | return NETDEV_TX_OK; | 1854 | return NETDEV_TX_OK; |
1855 | } | 1855 | } |