Messages in this thread | | | Date | Mon, 18 Jul 2022 14:03:45 +0300 | From | Leon Romanovsky <> | Subject | Re: [PATCH] RDMA/qedr: Fix potential memory leak in __qedr_alloc_mr() |
| |
On Thu, Jul 14, 2022 at 02:15:05PM +0800, Jianglei Nie wrote: > __qedr_alloc_mr() allocates a memory chunk for "mr->info.pbl_table" with > init_mr_info(). When rdma_alloc_tid() and rdma_register_tid() fail, "mr" > is released while "mr->info.pbl_table" is not released, which will lead > to a memory leak. > > We should release the "mr->info.pbl_table" with qedr_free_pbl() when error > occurs to fix the memory leak. > > Signed-off-by: Jianglei Nie <niejianglei2021@163.com> > --- > drivers/infiniband/hw/qedr/verbs.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) >
Added fixes line. Fixes: e0290cce6ac0 ("qedr: Add support for memory registeration verbs")
Thanks, applied.
| |