From: Catalin Marinas <catalin.marinas@arm.com>
Date: Wed, 8 Jul 2009 10:17:34 +0000 (+0100)
Subject: Free the memory allocated by memdup_user() in fs/sysfs/bin.c
X-Git-Tag: v3.0~8512
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d5ce5b40bc66880d1732461d4b47d7fc3331ed30;p=platform%2Fkernel%2Flinux-amlogic.git

Free the memory allocated by memdup_user() in fs/sysfs/bin.c

Commit 1c8542c7bb replaced kmalloc() with memdup_user() in the write()
function but also dropped the kfree(temp). The memdup_user() function
allocates memory which is never freed.

Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
Cc: Parag Warudkar <parag.warudkar@gmail.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
---

diff --git a/fs/sysfs/bin.c b/fs/sysfs/bin.c
index 9345806..2524714 100644
--- a/fs/sysfs/bin.c
+++ b/fs/sysfs/bin.c
@@ -171,6 +171,7 @@ static ssize_t write(struct file *file, const char __user *userbuf,
 	if (count > 0)
 		*off = offs + count;
 
+	kfree(temp);
 	return count;
 }