aboutsummaryrefslogtreecommitdiffstats
path: root/tools/bpf/bpftool/prog.c
diff options
context:
space:
mode:
authorQuentin Monnet <quentin.monnet@netronome.com>2017-10-19 18:46:20 -0400
committerDavid S. Miller <davem@davemloft.net>2017-10-21 21:11:32 -0400
commit1739c26da72c4170c86c368c75133adbb740efef (patch)
tree7442f029db148ae80dcb1b04850a6107211189ea /tools/bpf/bpftool/prog.c
parent9cbe1f581d17baff7e93936feb041c90b29eb6a8 (diff)
tools: bpftool: fix return value when all eBPF programs have been shown
Change the program to have a more consistent return code. Specifically, do not make bpftool return an error code simply because it reaches the end of the list of the eBPF programs to show. Signed-off-by: Quentin Monnet <quentin.monnet@netronome.com> Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com> Acked-by: Daniel Borkmann <daniel@iogearbox.net> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'tools/bpf/bpftool/prog.c')
-rw-r--r--tools/bpf/bpftool/prog.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/tools/bpf/bpftool/prog.c b/tools/bpf/bpftool/prog.c
index aa6d72ea3807..ede7957adcd9 100644
--- a/tools/bpf/bpftool/prog.c
+++ b/tools/bpf/bpftool/prog.c
@@ -275,8 +275,10 @@ static int do_show(int argc, char **argv)
275 while (true) { 275 while (true) {
276 err = bpf_prog_get_next_id(id, &id); 276 err = bpf_prog_get_next_id(id, &id);
277 if (err) { 277 if (err) {
278 if (errno == ENOENT) 278 if (errno == ENOENT) {
279 err = 0;
279 break; 280 break;
281 }
280 err("can't get next program: %s\n", strerror(errno)); 282 err("can't get next program: %s\n", strerror(errno));
281 if (errno == EINVAL) 283 if (errno == EINVAL)
282 err("kernel too old?\n"); 284 err("kernel too old?\n");