From: Robert-Ionut Alexa <robert-ionut.alexa@nxp.com>
Date: Fri, 23 Apr 2021 09:01:51 +0000 (+0300)
Subject: dpaa2-console.c: free buffer before returning from dpaa2_console_read
X-Git-Tag: rel_imx_5.10.35_2.0.0-somdevices.0~57
X-Git-Url: https://git.somdevices.com/?a=commitdiff_plain;h=12853a2d942d2c0982f5ea4f17a0c59128a3a363;p=linux.git

dpaa2-console.c: free buffer before returning from dpaa2_console_read

Dpaa2_console_read function frees kbuf buffer before returning.
The variable no longer goes out of scope, leaking the storage it
points to.

Signed-off-by: Robert-Ionut Alexa <robert-ionut.alexa@nxp.com>
---

diff --git a/drivers/soc/fsl/dpaa2-console.c b/drivers/soc/fsl/dpaa2-console.c
index 27243f706f37..e823fb9c9318 100644
--- a/drivers/soc/fsl/dpaa2-console.c
+++ b/drivers/soc/fsl/dpaa2-console.c
@@ -3,7 +3,7 @@
  * Freescale DPAA2 Platforms Console Driver
  *
  * Copyright 2015-2016 Freescale Semiconductor Inc.
- * Copyright 2018 NXP
+ * Copyright 2021 NXP
  */
 
 #define pr_fmt(fmt) "dpaa2-console: " fmt
@@ -231,6 +231,7 @@ static ssize_t dpaa2_console_read(struct file *fp, char __user *buf,
 	cd->cur_ptr += bytes;
 	written += bytes;
 
+	kfree(kbuf);
 	return written;
 
 err_free_buf: