diff options
author | Chen Gang <gang.chen@asianux.com> | 2013-04-08 12:06:44 +0800 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2013-04-08 13:26:56 +0200 |
commit | 67012ab1d2ce871afea4ee55408f233f97d09d07 (patch) | |
tree | 3f44d6c02233abb51848bfa991d571e86397b431 | |
parent | c97847d2f0eb77c806e650e04d9bbcf79fa05730 (diff) | |
download | blackbird-op-linux-67012ab1d2ce871afea4ee55408f233f97d09d07.tar.gz blackbird-op-linux-67012ab1d2ce871afea4ee55408f233f97d09d07.zip |
perf: Fix strncpy() use, use strlcpy() instead of strncpy()
For NUL terminated string we always need to set '\0' at the end.
Signed-off-by: Chen Gang <gang.chen@asianux.com>
Cc: rostedt@goodmis.org
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Link: http://lkml.kernel.org/r/51624254.30301@asianux.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
-rw-r--r-- | kernel/trace/trace.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index 4f1dade56981..3f5046a4d81b 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -132,7 +132,7 @@ static char *default_bootup_tracer; static int __init set_cmdline_ftrace(char *str) { - strncpy(bootup_tracer_buf, str, MAX_TRACER_SIZE); + strlcpy(bootup_tracer_buf, str, MAX_TRACER_SIZE); default_bootup_tracer = bootup_tracer_buf; /* We are using ftrace early, expand it */ ring_buffer_expanded = 1; @@ -162,7 +162,7 @@ static char *trace_boot_options __initdata; static int __init set_trace_boot_options(char *str) { - strncpy(trace_boot_options_buf, str, MAX_TRACER_SIZE); + strlcpy(trace_boot_options_buf, str, MAX_TRACER_SIZE); trace_boot_options = trace_boot_options_buf; return 0; } |