diff options
author | Steven Rostedt <srostedt@redhat.com> | 2009-05-08 16:06:47 -0400 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2009-05-08 16:06:47 -0400 |
commit | 29f93943d1916d1a3faa3f10f4a06994347ac990 (patch) | |
tree | 128c64e3e8305fb1c6785ef150b22b2412f856f2 /kernel/trace/trace_events.c | |
parent | c142b15dc56ee6d55cb97a062e3c8e9c61e384c0 (diff) |
tracing: initialize return value for __ftrace_set_clr_event
Commit 8f31bfe538ebafac187d2d4465a92e1d9ee6d8c2
tracing/events: clean up for ftrace_set_clr_event()
Moved out the code for ftrace_set_clr_event into a helper funciton but
did not initialize the return value. As a result, we do not warn about
a typo in the echoing of events in set_event.
This patch restores the old warning:
# echo foobar > set_event
-bash: echo: write error: Invalid argument
[ Impact: restore warning of invalid entries to set_event ]
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace/trace_events.c')
-rw-r--r-- | kernel/trace/trace_events.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/trace/trace_events.c b/kernel/trace/trace_events.c index df394bc6d54..2eecb87e42d 100644 --- a/kernel/trace/trace_events.c +++ b/kernel/trace/trace_events.c | |||
@@ -118,7 +118,7 @@ static int __ftrace_set_clr_event(const char *match, const char *sub, | |||
118 | const char *event, int set) | 118 | const char *event, int set) |
119 | { | 119 | { |
120 | struct ftrace_event_call *call; | 120 | struct ftrace_event_call *call; |
121 | int ret; | 121 | int ret = -EINVAL; |
122 | 122 | ||
123 | mutex_lock(&event_mutex); | 123 | mutex_lock(&event_mutex); |
124 | list_for_each_entry(call, &ftrace_events, list) { | 124 | list_for_each_entry(call, &ftrace_events, list) { |