summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSteven Rostedt (Red Hat) <rostedt@goodmis.org>2013-05-09 11:35:12 -0400
committerSteven Rostedt <rostedt@goodmis.org>2013-05-09 11:35:12 -0400
commit7c088b5120ffef017e2ddc38f992277e96436ef6 (patch)
tree9cfeea9e1adc62a0e0ee452bdcbb6fc748358b6d
parentff305ded9ff83436039a16d31bc558dc6598d7ce (diff)
downloadop-kernel-dev-7c088b5120ffef017e2ddc38f992277e96436ef6.zip
op-kernel-dev-7c088b5120ffef017e2ddc38f992277e96436ef6.tar.gz
ftrace: Have ftrace_regex_write() return either read or error
As ftrace_regex_write() reads the result of ftrace_process_regex() which can sometimes return a positive number, only consider a failure if the return is negative. Otherwise, it will skip possible other registered probes and by returning a positive number that wasn't read, it will confuse the user processes doing the writing. Cc: Masami Hiramatsu <masami.hiramatsu.pt@hitachi.com> Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
-rw-r--r--kernel/trace/ftrace.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c
index 8a5c017..d85a0ad 100644
--- a/kernel/trace/ftrace.c
+++ b/kernel/trace/ftrace.c
@@ -3276,7 +3276,7 @@ ftrace_regex_write(struct file *file, const char __user *ubuf,
ret = ftrace_process_regex(iter->hash, parser->buffer,
parser->idx, enable);
trace_parser_clear(parser);
- if (ret)
+ if (ret < 0)
goto out_unlock;
}
OpenPOWER on IntegriCloud