diff options
| author | Steven Rostedt (VMware) <rostedt@goodmis.org> | 2018-12-20 13:20:07 -0500 |
|---|---|---|
| committer | Steven Rostedt (VMware) <rostedt@goodmis.org> | 2018-12-22 22:51:54 -0500 |
| commit | b6b2735514bcd70ad1556a33892a636b20ece671 (patch) | |
| tree | a51e1477fef62ca3223c272306ab4162ae23cb05 /kernel/trace/trace_events.c | |
| parent | 754481e6954cbef53f8bc4412ad48dde611e21d3 (diff) | |
| download | linux-next-b6b2735514bcd70ad1556a33892a636b20ece671.tar.gz | |
tracing: Use str_has_prefix() instead of using fixed sizes
There are several instances of strncmp(str, "const", 123), where 123 is the
strlen of the const string to check if "const" is the prefix of str. But
this can be error prone. Use str_has_prefix() instead.
Acked-by: Namhyung Kim <namhyung@kernel.org>
Signed-off-by: Steven Rostedt (VMware) <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 bd0162c0467c..5b3b0c3c8a47 100644 --- a/kernel/trace/trace_events.c +++ b/kernel/trace/trace_events.c @@ -1251,7 +1251,7 @@ static int f_show(struct seq_file *m, void *v) */ array_descriptor = strchr(field->type, '['); - if (!strncmp(field->type, "__data_loc", 10)) + if (str_has_prefix(field->type, "__data_loc")) array_descriptor = NULL; if (!array_descriptor) |