Re: [PATCH net] nfp: freeing the wrong variable

2017-07-12 Thread David Miller
From: Dan Carpenter Date: Wed, 12 Jul 2017 10:42:06 +0300 > We accidentally free a NULL pointer and leak the pointer we want to > free. Also you can tell from the label name what was intended. :) > > Fixes: abfcdc1de9bf ("nfp: add a stats handler for flower offloads") > Signed-off-by: Dan Carp

Re: [PATCH net] nfp: freeing the wrong variable

2017-07-12 Thread Jakub Kicinski
On Wed, 12 Jul 2017 10:42:06 +0300, Dan Carpenter wrote: > We accidentally free a NULL pointer and leak the pointer we want to > free. Also you can tell from the label name what was intended. :) > > Fixes: abfcdc1de9bf ("nfp: add a stats handler for flower offloads") > Signed-off-by: Dan Carpent

[PATCH net] nfp: freeing the wrong variable

2017-07-12 Thread Dan Carpenter
We accidentally free a NULL pointer and leak the pointer we want to free. Also you can tell from the label name what was intended. :) Fixes: abfcdc1de9bf ("nfp: add a stats handler for flower offloads") Signed-off-by: Dan Carpenter diff --git a/drivers/net/ethernet/netronome/nfp/flower/metadat