diff options
author | Li Zefan <lizf@cn.fujitsu.com> | 2009-01-14 20:30:52 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-01-15 05:28:32 -0500 |
commit | 42fab4b2cdc02cf28e2474ccfd75bc9225076590 (patch) | |
tree | 22a620e154b2d73df89747a2d0b3bf6efc2f064b | |
parent | 4a2b8dda3f8705880ec7408135645602d5590f51 (diff) |
tracing/ftrace: add missing unlock in register_stat_tracer()
We should unlock all_stat_sessions_mutex before returning failure.
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Signed-off-by: Steven Rostedt <srostedt@redhat.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r-- | kernel/trace/trace_stat.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/kernel/trace/trace_stat.c b/kernel/trace/trace_stat.c index cb29282b9485..2110cea2ece3 100644 --- a/kernel/trace/trace_stat.c +++ b/kernel/trace/trace_stat.c | |||
@@ -73,8 +73,10 @@ int register_stat_tracer(struct tracer_stat *trace) | |||
73 | /* Already registered? */ | 73 | /* Already registered? */ |
74 | mutex_lock(&all_stat_sessions_mutex); | 74 | mutex_lock(&all_stat_sessions_mutex); |
75 | list_for_each_entry_safe(node, tmp, &all_stat_sessions, session_list) { | 75 | list_for_each_entry_safe(node, tmp, &all_stat_sessions, session_list) { |
76 | if (node->ts == trace) | 76 | if (node->ts == trace) { |
77 | mutex_unlock(&all_stat_sessions_mutex); | ||
77 | return -EINVAL; | 78 | return -EINVAL; |
79 | } | ||
78 | } | 80 | } |
79 | mutex_unlock(&all_stat_sessions_mutex); | 81 | mutex_unlock(&all_stat_sessions_mutex); |
80 | 82 | ||