diff options
author | Frederic Weisbecker <fweisbec@gmail.com> | 2009-10-14 14:43:39 -0400 |
---|---|---|
committer | Frederic Weisbecker <fweisbec@gmail.com> | 2009-10-14 14:43:39 -0400 |
commit | 06f43d66ec36388056f5c697bf1e67c0e0a1645c (patch) | |
tree | 04b8735f670570d146c0c12bb403effc94e14346 | |
parent | 459c6d15a0c52bae43842ff2cd0dd41aa7de9b7f (diff) |
ftrace: Copy ftrace_graph_filter boot param using strlcpy
We are using strncpy in the wrong way to copy the ftrace_graph_filter
boot param because we pass the buffer size instead of the max string
size it can contain (buffer size - 1). The end result might not be
NULL terminated as we are abusing the max string size.
Lets use strlcpy() instead.
Reported-by: Li Zefan <lizf@cn.fujitsu.com>
Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Steven Rostedt <rostedt@goodmis.org>
-rw-r--r-- | kernel/trace/ftrace.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index aaea9cda8781..b10c0d90a6ff 100644 --- a/kernel/trace/ftrace.c +++ b/kernel/trace/ftrace.c | |||
@@ -2293,7 +2293,7 @@ __setup("ftrace_filter=", set_ftrace_filter); | |||
2293 | #ifdef CONFIG_FUNCTION_GRAPH_TRACER | 2293 | #ifdef CONFIG_FUNCTION_GRAPH_TRACER |
2294 | static int __init set_graph_function(char *str) | 2294 | static int __init set_graph_function(char *str) |
2295 | { | 2295 | { |
2296 | strncpy(ftrace_graph_buf, str, FTRACE_FILTER_SIZE); | 2296 | strlcpy(ftrace_graph_buf, str, FTRACE_FILTER_SIZE); |
2297 | return 1; | 2297 | return 1; |
2298 | } | 2298 | } |
2299 | __setup("ftrace_graph_filter=", set_graph_function); | 2299 | __setup("ftrace_graph_filter=", set_graph_function); |