Re: [PATCH] fm10k: fix memory leak

2015-11-18 Thread Eric Dumazet
On Tue, 2015-11-17 at 14:24 -0800, Stephen Hemminger wrote: > This was detected by Coverity. > The function skb_cow_head leaves skb alone on failure, so caller needs > to free. > > Signed-off-by: Stephen Hemminger > > --- a/drivers/net/ethernet/intel/fm10k/fm10k_netdev.c 2015-11-17 > 13:33:40.0

Re: [PATCH] fm10k: fix memory leak

2015-11-18 Thread Jeff Kirsher
On Wed, 2015-11-18 at 15:06 -0500, David Miller wrote: > From: Stephen Hemminger > Date: Tue, 17 Nov 2015 14:24:27 -0800 > > > This was detected by Coverity. > > The function skb_cow_head leaves skb alone on failure, so caller > > needs > > to free. > > > > Signed-off-by: Stephen Hemminger > >

Re: [PATCH] fm10k: fix memory leak

2015-11-18 Thread David Miller
From: Stephen Hemminger Date: Tue, 17 Nov 2015 14:24:27 -0800 > This was detected by Coverity. > The function skb_cow_head leaves skb alone on failure, so caller needs > to free. > > Signed-off-by: Stephen Hemminger Applied. -- To unsubscribe from this list: send the line "unsubscribe netdev"

[PATCH] fm10k: fix memory leak

2015-11-17 Thread Stephen Hemminger
This was detected by Coverity. The function skb_cow_head leaves skb alone on failure, so caller needs to free. Signed-off-by: Stephen Hemminger --- a/drivers/net/ethernet/intel/fm10k/fm10k_netdev.c 2015-11-17 13:33:40.032694657 -0800 +++ b/drivers/net/ethernet/intel/fm10k/fm10k_netdev.c 201