Re: [PATCH 2/2] IB/sysfs: fix memory leak

2016-06-07 Thread Doug Ledford
On 6/5/2016 4:13 PM, Sudip Mukherjee wrote: > If the allocation of hsag fails we were just returning but stats was > not released. > > Signed-off-by: Sudip Mukherjee > --- > drivers/infiniband/core/sysfs.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/infiniba

Re: [PATCH 2/2] IB/sysfs: fix memory leak

2016-06-06 Thread Leon Romanovsky
On Mon, Jun 06, 2016 at 09:34:54AM +0300, Leon Romanovsky wrote: > On Sun, Jun 05, 2016 at 09:13:56PM +0100, Sudip Mukherjee wrote: > > If the allocation of hsag fails we were just returning but stats was > > not released. > > > > Signed-off-by: Sudip Mukherjee > > Thanks, > Reviewed-by: Leon Ro

Re: [PATCH 2/2] IB/sysfs: fix memory leak

2016-06-05 Thread Leon Romanovsky
On Sun, Jun 05, 2016 at 09:13:56PM +0100, Sudip Mukherjee wrote: > If the allocation of hsag fails we were just returning but stats was > not released. > > Signed-off-by: Sudip Mukherjee Thanks, Reviewed-by: Leon Romanovsky > --- > drivers/infiniband/core/sysfs.c | 2 +- > 1 file changed, 1 i

[PATCH 2/2] IB/sysfs: fix memory leak

2016-06-05 Thread Sudip Mukherjee
If the allocation of hsag fails we were just returning but stats was not released. Signed-off-by: Sudip Mukherjee --- drivers/infiniband/core/sysfs.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/infiniband/core/sysfs.c b/drivers/infiniband/core/sysfs.c index fcf6f9