From eb60e7f1f1c30f250d7e6214131f5280185b208c Mon Sep 17 00:00:00 2001 From: Shashank Singh Date: Mon, 5 Dec 2022 08:39:06 +0000 Subject: [PATCH] gpu: nvgpu: fix memory leak in pd cache Fix memory leak in case dma allocation fails in function nvgpu_pd_cache_alloc_direct_lock(). This one got introduced by the patch https://git-master.nvidia.com/r/c/linux-nvgpu/+/2735657. Bug 3461002 Change-Id: Ia8c6138179cc9bb2adc53bff32dd81fb2b6ee54a Signed-off-by: Shashank Singh Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvgpu/+/2821684 Reviewed-by: Sachin Nikam Reviewed-by: svcacv Reviewed-by: svc-mobile-coverity Reviewed-by: svc-mobile-misra Reviewed-by: Sagar Kamble Reviewed-by: Rajesh Devaraj Reviewed-by: Ankur Kishore GVS: Gerrit_Virtual_Submit --- drivers/gpu/nvgpu/common/mm/gmmu/pd_cache.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/nvgpu/common/mm/gmmu/pd_cache.c b/drivers/gpu/nvgpu/common/mm/gmmu/pd_cache.c index ed8db0760..07d422f98 100644 --- a/drivers/gpu/nvgpu/common/mm/gmmu/pd_cache.c +++ b/drivers/gpu/nvgpu/common/mm/gmmu/pd_cache.c @@ -228,7 +228,7 @@ static int nvgpu_pd_cache_alloc_direct_locked(struct gk20a *g, err = nvgpu_dma_alloc_flags(g, flags, bytes, &(pentry->mem)); if (err != 0) { nvgpu_err(g, "OOM allocating page directory!"); - nvgpu_kfree(g, pd->mem); + nvgpu_kfree(g, pentry); return -ENOMEM; }