From 35393d7615c325d06a595cbbe3dc633991519d06 Mon Sep 17 00:00:00 2001 From: Prateek Patel Date: Wed, 12 Jan 2022 09:06:28 +0000 Subject: [PATCH] drivers: adsp: fix resource leak Coverity defect: Variable msg_recv going out of scope leaks the storage it points to. Fix the defect by freeing msg_recv before return. CID 31655 Bug 3461002 Change-Id: I350191943d342d33b1d07155ddcc2d875a78f7af Signed-off-by: Prateek Patel Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2653460 Reviewed-by: svcacv Reviewed-by: svc_kernel_abi Reviewed-by: Sachin Nikam Reviewed-by: mobile promotions GVS: Gerrit_Virtual_Submit Tested-by: mobile promotions --- drivers/platform/tegra/nvadsp/adspff.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/platform/tegra/nvadsp/adspff.c b/drivers/platform/tegra/nvadsp/adspff.c index bf9148c9..7d971788 100644 --- a/drivers/platform/tegra/nvadsp/adspff.c +++ b/drivers/platform/tegra/nvadsp/adspff.c @@ -387,6 +387,7 @@ void adspff_fwrite(void) (msgq_message_t *)&message); if (ret < 0) { pr_err("fwrite Dequeue failed %d.", ret); + kfree(msg_recv); return; }