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 /kernel/trace | |
| 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>
Diffstat (limited to 'kernel/trace')
| -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 | ||
