Re: Re: [PATCH] IB/uverbs: Fix memleak in ib_uverbs_add_one

2020-08-21 Thread dinghao . liu
> > > On 21 Aug 2020, at 10:10, Dinghao Liu wrote: > > > > When ida_alloc_max() fails, uverbs_dev should be freed > > just like when init_srcu_struct() fails. It's the same > > for the error paths after this call. > > > > Signed-off-by: Dinghao Liu > > --- > > drivers/infiniband/core/uverbs_ma

Re: [PATCH] IB/uverbs: Fix memleak in ib_uverbs_add_one

2020-08-21 Thread Jason Gunthorpe
On Fri, Aug 21, 2020 at 11:47:32AM +0200, Håkon Bugge wrote: > > > > On 21 Aug 2020, at 10:10, Dinghao Liu wrote: > > > > When ida_alloc_max() fails, uverbs_dev should be freed > > just like when init_srcu_struct() fails. It's the same > > for the error paths after this call. > > > > Signed-of

Re: [PATCH] IB/uverbs: Fix memleak in ib_uverbs_add_one

2020-08-21 Thread Håkon Bugge
> On 21 Aug 2020, at 10:10, Dinghao Liu wrote: > > When ida_alloc_max() fails, uverbs_dev should be freed > just like when init_srcu_struct() fails. It's the same > for the error paths after this call. > > Signed-off-by: Dinghao Liu > --- > drivers/infiniband/core/uverbs_main.c | 1 + > 1 fil

[PATCH] IB/uverbs: Fix memleak in ib_uverbs_add_one

2020-08-21 Thread Dinghao Liu
When ida_alloc_max() fails, uverbs_dev should be freed just like when init_srcu_struct() fails. It's the same for the error paths after this call. Signed-off-by: Dinghao Liu --- drivers/infiniband/core/uverbs_main.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/infiniband/core/uver