Commit 15351d5e authored Apr 23, 2021 by Robert-Ionut Alexa Committed by Greg Kroah-Hartman Nov 17, 2021

soc: fsl: dpaa2-console: free buffer before returning from dpaa2_console_read



[ Upstream commit 8120bd469f5525da229953c1197f2b826c0109f4 ]

Free the kbuf buffer before returning from the dpaa2_console_read()
function. The variable no longer goes out of scope, leaking the storage
it points to.

Fixes: c93349d8 ("soc: fsl: add DPAA2 console support")
Signed-off-by: Robert-Ionut Alexa <robert-ionut.alexa@nxp.com>
Signed-off-by: Ioana Ciornei <ioana.ciornei@nxp.com>
Signed-off-by: Li Yang <leoyang.li@nxp.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>

parent be832f78

drivers/soc/fsl/dpaa2-console.c

+1 −0

Original line number	Diff line number	Diff line
		@@ -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: