diff options
author | Jiri Slaby <jslaby@suse.cz> | 2010-10-10 23:26:56 (GMT) |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-10-11 18:13:59 (GMT) |
commit | 419c20465d6319274e7286f0966e2390540e6e0a (patch) | |
tree | c4fca8f173fdd69b07a08455b34e0c01aacbe949 /drivers/net/pch_gbe/pch_gbe_main.c | |
parent | 34d101dd6204bd100fc2e6f7b5f9a10f959ce2c9 (diff) | |
download | linux-fsl-qoriq-419c20465d6319274e7286f0966e2390540e6e0a.tar.xz |
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/pch_gbe_main.c')
-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 53c56cf..e44644f 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) unsigned long flags; if (unlikely(skb->len > (adapter->hw.mac.max_frame_size - 4))) { - dev_kfree_skb_any(skb); pr_err("Transfer length Error: skb len: %d > max: %d\n", skb->len, adapter->hw.mac.max_frame_size); + dev_kfree_skb_any(skb); adapter->stats.tx_length_errors++; return NETDEV_TX_OK; } |