summaryrefslogtreecommitdiffstats
path: root/security
diff options
context:
space:
mode:
authorXiaochen Wang <wangxiaochen0@gmail.com>2011-03-31 00:27:32 +0900
committerJames Morris <jmorris@namei.org>2011-03-31 10:25:06 +1100
commitcfc64fd91fabed099a4c3df58559f4b7efe9bcce (patch)
tree6ecc7efb4d61ba9e1ae21a11e50f175961f76cf8 /security
parent93b9c98b3498db5842e2812b32cff4c1ae947eb1 (diff)
downloadlinux-cfc64fd91fabed099a4c3df58559f4b7efe9bcce.tar.gz
linux-cfc64fd91fabed099a4c3df58559f4b7efe9bcce.tar.bz2
linux-cfc64fd91fabed099a4c3df58559f4b7efe9bcce.zip
tomoyo: fix memory leak in tomoyo_commit_ok()
When memory used for policy exceeds the quota, tomoyo_memory_ok() return false. In this case, tomoyo_commit_ok() must call kfree() before returning NULL. This bug exists since 2.6.35. Signed-off-by: Xiaochen Wang <wangxiaochen0@gmail.com> Acked-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp> Signed-off-by: James Morris <jmorris@namei.org>
Diffstat (limited to 'security')
-rw-r--r--security/tomoyo/memory.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/security/tomoyo/memory.c b/security/tomoyo/memory.c
index 297612669c74..42a7b1ba8cbf 100644
--- a/security/tomoyo/memory.c
+++ b/security/tomoyo/memory.c
@@ -75,6 +75,7 @@ void *tomoyo_commit_ok(void *data, const unsigned int size)
memset(data, 0, size);
return ptr;
}
+ kfree(ptr);
return NULL;
}