diff options
author | Li Zefan <lizf@cn.fujitsu.com> | 2009-09-22 01:54:28 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-09-22 04:28:57 -0400 |
commit | 79fe249c8368be35c9ca05982e80c68e959505e1 (patch) | |
tree | cfb0b12a3d189fde435db4cab817b740c14517f3 /kernel/trace | |
parent | 1eb90f138b3fb4cc15f3acec94aa788e846269f7 (diff) |
tracing: Fix failure path in ftrace_regex_open()
Don't forget to free trace_parser if seq_open() returned failure.
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
LKML-Reference: <4AB86694.4040803@cn.fujitsu.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/trace')
-rw-r--r-- | kernel/trace/ftrace.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index 5c5cb9be8e8c..e70af98bb99c 100644 --- a/kernel/trace/ftrace.c +++ b/kernel/trace/ftrace.c | |||
@@ -1621,8 +1621,10 @@ ftrace_regex_open(struct inode *inode, struct file *file, int enable) | |||
1621 | if (!ret) { | 1621 | if (!ret) { |
1622 | struct seq_file *m = file->private_data; | 1622 | struct seq_file *m = file->private_data; |
1623 | m->private = iter; | 1623 | m->private = iter; |
1624 | } else | 1624 | } else { |
1625 | trace_parser_put(&iter->parser); | ||
1625 | kfree(iter); | 1626 | kfree(iter); |
1627 | } | ||
1626 | } else | 1628 | } else |
1627 | file->private_data = iter; | 1629 | file->private_data = iter; |
1628 | mutex_unlock(&ftrace_regex_lock); | 1630 | mutex_unlock(&ftrace_regex_lock); |