summaryrefslogtreecommitdiffstats
path: root/kernel
diff options
context:
space:
mode:
authorJulia Lawall <julia@diku.dk>2008-11-14 19:05:31 +0100
committerIngo Molnar <mingo@elte.hu>2008-11-18 16:59:58 +0100
commit0bb943c7a2136716757a263f604d26309fd98042 (patch)
tree80af655ff3272a820a9090e6fa08c9e6472f0bed /kernel
parente270219f4372b58bd3eeac12bd9f7edc592b8f6b (diff)
downloadop-kernel-dev-0bb943c7a2136716757a263f604d26309fd98042.zip
op-kernel-dev-0bb943c7a2136716757a263f604d26309fd98042.tar.gz
tracing: kernel/trace/trace.c: introduce missing kfree()
Impact: fix memory leak Error handling code following a kzalloc should free the allocated data. The semantic match that finds the problem is as follows: (http://www.emn.fr/x-info/coccinelle/) // <smpl> @r exists@ local idexpression x; statement S; expression E; identifier f,l; position p1,p2; expression *ptr != NULL; @@ ( if ((x@p1 = \(kmalloc\|kzalloc\|kcalloc\)(...)) == NULL) S | x@p1 = \(kmalloc\|kzalloc\|kcalloc\)(...); ... if (x == NULL) S ) <... when != x when != if (...) { <+...x...+> } x->f = E ...> ( return \(0\|<+...x...+>\|ptr\); | return@p2 ...; ) @script:python@ p1 << r.p1; p2 << r.p2; @@ print "* file: %s kmalloc %s return %s" % (p1[0].file,p1[0].line,p2[0].line) // </smpl> Signed-off-by: Julia Lawall <julia@diku.dk> Signed-off-by: Steven Rostedt <srostedt@redhat.com> Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel')
-rw-r--r--kernel/trace/trace.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c
index 697eda3..d86e325 100644
--- a/kernel/trace/trace.c
+++ b/kernel/trace/trace.c
@@ -1936,6 +1936,7 @@ __tracing_open(struct inode *inode, struct file *file, int *ret)
ring_buffer_read_finish(iter->buffer_iter[cpu]);
}
mutex_unlock(&trace_types_lock);
+ kfree(iter);
return ERR_PTR(-ENOMEM);
}
OpenPOWER on IntegriCloud