diff options
author | Li RongQing <roy.qing.li@gmail.com> | 2014-12-19 05:19:35 (GMT) |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-12-19 18:19:45 (GMT) |
commit | 4f2ff8ef9eab32e6bc01c71bb539fc68d0917f59 (patch) | |
tree | 6736c3b45b18f7eba8f24bbf6a3666dd4f51216e /drivers/net | |
parent | d790be3863b28fd22e0781c1a3ddefcbfd5f7086 (diff) | |
download | linux-4f2ff8ef9eab32e6bc01c71bb539fc68d0917f59.tar.xz |
sunvnet: fix a memory leak in vnet_handle_offloads
when skb_gso_segment returns error, the original skb should be freed
Signed-off-by: Li RongQing <roy.qing.li@gmail.com>
Acked-by: David L Stevens <david.stevens@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/ethernet/sun/sunvnet.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/net/ethernet/sun/sunvnet.c b/drivers/net/ethernet/sun/sunvnet.c index 45c408e..d2835bf 100644 --- a/drivers/net/ethernet/sun/sunvnet.c +++ b/drivers/net/ethernet/sun/sunvnet.c @@ -1201,6 +1201,7 @@ static int vnet_handle_offloads(struct vnet_port *port, struct sk_buff *skb) segs = skb_gso_segment(skb, dev->features & ~NETIF_F_TSO); if (IS_ERR(segs)) { dev->stats.tx_dropped++; + dev_kfree_skb_any(skb); return NETDEV_TX_OK; } |