[PATCH] staging: android: fix a possible memory leak

From: Seunghun Lee
Date: Wed Aug 13 2014 - 13:03:03 EST


Memory allocated by kstrdup should be freed.

CC: Brian Swetland <swetland@xxxxxxxxxx>
Signed-off-by: Seunghun Lee <waydi1@xxxxxxxxx>
---
drivers/staging/android/logger.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/android/logger.c b/drivers/staging/android/logger.c
index 9b47e66..0bf0d24 100644
--- a/drivers/staging/android/logger.c
+++ b/drivers/staging/android/logger.c
@@ -790,7 +790,7 @@ static int __init create_log(char *log_name, int size)
if (unlikely(ret)) {
pr_err("failed to register misc device for log '%s'!\n",
log->misc.name);
- goto out_free_log;
+ goto out_free_misc_name;
}

pr_info("created %luK log '%s'\n",
@@ -798,6 +798,9 @@ static int __init create_log(char *log_name, int size)

return 0;

+out_free_misc_name:
+ kfree(log->misc.name);
+
out_free_log:
kfree(log);

--
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/