Re: [patch net-next-2.6 v2] skbuff: fix error handling in pskb_copy()

[patch] skbuff: use kfree_skb() instead of kfree() · Dan Carpenter <hidden> · 2011-07-20
Re: [patch] skbuff: use kfree_skb() instead of kfree() · Dan Carpenter <hidden> · 2011-07-20
Re: [patch] skbuff: use kfree_skb() instead of kfree() · Eric Dumazet <hidden> · 2011-07-20
Re: [patch] skbuff: use kfree_skb() instead of kfree() · Dan Carpenter <hidden> · 2011-07-20
Re: [patch] skbuff: use kfree_skb() instead of kfree() · Eric Dumazet <hidden> · 2011-07-20
Re: [patch] skbuff: use kfree_skb() instead of kfree() · Julia Lawall <hidden> · 2011-07-20
Re: [patch] skbuff: use kfree_skb() instead of kfree() · Julie Sullivan <hidden> · 2011-07-20
[patch net-next-2.6 v2] skbuff: fix error handling in pskb_copy() · Dan Carpenter <hidden> · 2011-07-20
Re: [patch net-next-2.6 v2] skbuff: fix error handling in pskb_copy() · Eric Dumazet <hidden> · 2011-07-20
Re: [patch net-next-2.6 v2] skbuff: fix error handling in pskb_copy() · David Miller <davem@davemloft.net> · 2011-07-21

From: Eric Dumazet <hidden>
Date: 2011-07-20 08:59:55
Also in: kernel-janitors

Le mercredi 20 juillet 2011 à 11:51 +0300, Dan Carpenter a écrit :

There are two problems:
1) "n" was allocated with alloc_skb() so we should free it with
   kfree_skb() instead of regular kfree().
2) We return the freed pointer instead of NULL.

Signed-off-by: Dan Carpenter <redacted>

Thanks Dan

Reviewed-by: Eric Dumazet <redacted>

`h`	back out one level
`j`	next message in thread
`k`	previous message in thread
`l`	drill in
`Esc`	close help / fold thread tree
`?`	toggle this help