diff options
author | Quentin Monnet <quentin.monnet@netronome.com> | 2019-08-15 15:32:19 +0100 |
---|---|---|
committer | Alexei Starovoitov <ast@kernel.org> | 2019-08-15 22:06:46 -0700 |
commit | b0ead6d75a5b335287337e602e6b815e1115481c (patch) | |
tree | d14607a2eca6ae72c5342a1762229e8d1ec768c8 /tools/bpf/bpftool | |
parent | 8a15d5ced8c626c0331974c7281c1d651f7b0d83 (diff) | |
download | linux-b0ead6d75a5b335287337e602e6b815e1115481c.tar.bz2 |
tools: bpftool: fix format string for p_err() in detect_common_prefix()
There is one call to the p_err() function in detect_common_prefix()
where the message to print is passed directly as the first argument,
without using a format string. This is harmless, but may trigger
warnings if the "__printf()" attribute is used correctly for the p_err()
function. Let's fix it by using a "%s" format string.
Fixes: ba95c7452439 ("tools: bpftool: add "prog run" subcommand to test-run programs")
Signed-off-by: Quentin Monnet <quentin.monnet@netronome.com>
Reviewed-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Diffstat (limited to 'tools/bpf/bpftool')
-rw-r--r-- | tools/bpf/bpftool/main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/bpf/bpftool/main.c b/tools/bpf/bpftool/main.c index e916ff25697f..93d008687020 100644 --- a/tools/bpf/bpftool/main.c +++ b/tools/bpf/bpftool/main.c @@ -139,7 +139,7 @@ int detect_common_prefix(const char *arg, ...) strncat(msg, "'", sizeof(msg) - strlen(msg) - 1); if (count >= 2) { - p_err(msg); + p_err("%s", msg); return -1; } |