commit aa94fbd5ccd840c8ab26d02439ec799b03a72547
author Dan Carpenter <error27@gmail.com> Thu Oct 02 14:50:14 2008 -0700
committer Linus Torvalds <torvalds@linux-foundation.org> Thu Oct 02 15:53:13 2008 -0700
tree 37ebc732cc75d827e5779f2297b6a28c0f0eadaa
parent c0c9209ddd96bc4f1d70a8b9958710671e076080

fix error-path NULL deref in alloc_posix_timer()

Found by static checker (http://repo.or.cz/w/smatch.git).

Signed-off-by: Dan Carpenter <error27@gmail.com>
Acked-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>

kernel/posix-timers.c

diff --git a/kernel/posix-timers.c b/kernel/posix-timers.c
index e36d579..5131e54 100644
--- a/kernel/posix-timers.c
+++ b/kernel/posix-timers.c
@@ -441,7 +441,7 @@
 		return tmr;
 	if (unlikely(!(tmr->sigq = sigqueue_alloc()))) {
 		kmem_cache_free(posix_timers_cache, tmr);
-		tmr = NULL;
+		return NULL;
 	}
 	memset(&tmr->sigq->info, 0, sizeof(siginfo_t));
 	return tmr;