From 47abcf5d5fd4e6d187d63e695982ea9d85fffb92 Mon Sep 17 00:00:00 2001 From: Ella Feng Date: Thu, 24 Oct 2019 23:10:15 +0800 Subject: [PATCH] MGS-5176 [#imx-1744] Coveriy - need free the pages if malloc memory failed When alloc NonContiguous1MPages, if malloc fail, need free the pages to avoid memory leak Signed-off-by: Xianzhong Li (cherry picked from commit 30b05a615613a7e47db5e98dddb0fae9ca42b924) --- .../kernel/allocator/default/gc_hal_kernel_allocator_gfp.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/mxc/gpu-viv/hal/os/linux/kernel/allocator/default/gc_hal_kernel_allocator_gfp.c b/drivers/mxc/gpu-viv/hal/os/linux/kernel/allocator/default/gc_hal_kernel_allocator_gfp.c index b157252c5780..059512a10f22 100644 --- a/drivers/mxc/gpu-viv/hal/os/linux/kernel/allocator/default/gc_hal_kernel_allocator_gfp.c +++ b/drivers/mxc/gpu-viv/hal/os/linux/kernel/allocator/default/gc_hal_kernel_allocator_gfp.c @@ -424,6 +424,7 @@ _NonContiguous1MPagesAlloc( if (order >= MAX_ORDER) { + kfree(pages); _NonContiguous1MPagesFree(MdlPriv, MdlPriv->numPages1M); return gcvNULL; } @@ -433,6 +434,7 @@ _NonContiguous1MPagesAlloc( if (MdlPriv->Pages1M[i] == gcvNULL) { + kfree(pages); _NonContiguous1MPagesFree(MdlPriv, MdlPriv->numPages1M); return gcvNULL; } -- 2.17.1