Re: [PATCH v2 net] net: qualcomm: rmnet: Fix a double free

2017-09-09 Thread David Miller
From: Dan Carpenter Date: Sat, 9 Sep 2017 11:58:03 +0300 > There is a typo here so we accidentally free "skb" instead of "skbn". > It leads to a double free and a leak. After discussing with Subash, > it's better to just move the check before the allocation and avoid

Re: [PATCH v2 net] net: qualcomm: rmnet: Fix a double free

2017-09-09 Thread Subash Abhinov Kasiviswanathan
On 2017-09-09 02:58, Dan Carpenter wrote: There is a typo here so we accidentally free "skb" instead of "skbn". It leads to a double free and a leak. After discussing with Subash, it's better to just move the check before the allocation and avoid the need to free. Fixes: ceed73a2cf4a

[PATCH v2 net] net: qualcomm: rmnet: Fix a double free

2017-09-09 Thread Dan Carpenter
There is a typo here so we accidentally free "skb" instead of "skbn". It leads to a double free and a leak. After discussing with Subash, it's better to just move the check before the allocation and avoid the need to free. Fixes: ceed73a2cf4a ("drivers: net: ethernet: qualcomm: rmnet: Initial