projects
/
sdk
/
emulator
/
emulator-kernel.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
bca14dd
)
tomoyo: fix potential use after free
author
Dan Carpenter
<error27@gmail.com>
Sat, 13 Mar 2010 11:14:22 +0000
(14:14 +0300)
committer
James Morris
<jmorris@namei.org>
Sun, 14 Mar 2010 20:51:29 +0000
(07:51 +1100)
The original code returns a freed pointer. This function is expected to
return NULL on errors.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Signed-off-by: James Morris <jmorris@namei.org>
security/tomoyo/common.c
patch
|
blob
|
history
diff --git
a/security/tomoyo/common.c
b/security/tomoyo/common.c
index
ff51f10
..
ef89947
100644
(file)
--- a/
security/tomoyo/common.c
+++ b/
security/tomoyo/common.c
@@
-886,6
+886,7
@@
static struct tomoyo_profile *tomoyo_find_or_assign_new_profile(const unsigned
ptr = kmalloc(sizeof(*ptr), GFP_KERNEL);
if (!tomoyo_memory_ok(ptr)) {
kfree(ptr);
+ ptr = NULL;
goto ok;
}
for (i = 0; i < TOMOYO_MAX_CONTROL_INDEX; i++)