[PATCH v1] drivers:testing:Handle possible memory leaks

Yang Ruibin
2024-08-22T03:21:23Z

When copy_from_user() fails, -EFAULT is returned without
releasing the memory previously allocated by kmalloc().

Signed-off-by: Yang Ruibin <11162571@vivo.com>
---
drivers/thermal/testing/command.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/thermal/testing/command.c b/drivers/thermal/testing/command.c
index 6771eb7b1..7868caee3 100644
--- a/drivers/thermal/testing/command.c
+++ b/drivers/thermal/testing/command.c
@@ -151,6 +151,7 @@ static ssize_t tt_command_process(struct dentry *dentry, const char __user *user
return -ENOMEM;

if (copy_from_user(buf, user_buf, count))
+ kfree(buf);
return -EFAULT;

buf[count] = '\0';
--
2.34.1