diff options
author | Li Zefan <lizf@cn.fujitsu.com> | 2009-09-22 13:51:54 +0800 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-09-22 10:28:53 +0200 |
commit | 3c235a337e205da0f614e456be72881483dcde6e (patch) | |
tree | 73dee892e0cf269793c2193e235d9701fa9a3115 /kernel/trace | |
parent | b8c7f1dc5ca4e0d10709182233cdab932cef593d (diff) |
tracing: Fix off-by-one in trace_get_user()
Leave the last slot for the tailing '\0'.
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: <4AB865FA.5080801@cn.fujitsu.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/trace')
-rw-r--r-- | kernel/trace/trace.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index a35925d222b..ae17453dc0f 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -415,7 +415,7 @@ int trace_get_user(struct trace_parser *parser, const char __user *ubuf, /* read the non-space input */ while (cnt && !isspace(ch)) { - if (parser->idx < parser->size) + if (parser->idx < parser->size - 1) parser->buffer[parser->idx++] = ch; else { ret = -EINVAL; |