diff options
author | Quentin Monnet <quentin.monnet@netronome.com> | 2019-08-15 17:22:23 +0300 |
---|---|---|
committer | Alexei Starovoitov <ast@kernel.org> | 2019-08-16 08:09:41 +0300 |
commit | d34b044038bfb0e19caa8b019910efc465f41d5f (patch) | |
tree | 6972181b15474d393f514cfdd4a8b24d105201e7 /tools | |
parent | 27df5c7068bf23cab282dc64b1c9894429b3b8a0 (diff) | |
download | linux-d34b044038bfb0e19caa8b019910efc465f41d5f.tar.xz |
tools: bpftool: close prog FD before exit on showing a single program
When showing metadata about a single program by invoking
"bpftool prog show PROG", the file descriptor referring to the program
is not closed before returning from the function. Let's close it.
Fixes: 71bb428fe2c1 ("tools: bpf: add bpftool")
Signed-off-by: Quentin Monnet <quentin.monnet@netronome.com>
Reviewed-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Acked-by: Andrii Nakryiko <andriin@fb.com>
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Diffstat (limited to 'tools')
-rw-r--r-- | tools/bpf/bpftool/prog.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/tools/bpf/bpftool/prog.c b/tools/bpf/bpftool/prog.c index 66f04a4846a5..43fdbbfe41bb 100644 --- a/tools/bpf/bpftool/prog.c +++ b/tools/bpf/bpftool/prog.c @@ -363,7 +363,9 @@ static int do_show(int argc, char **argv) if (fd < 0) return -1; - return show_prog(fd); + err = show_prog(fd); + close(fd); + return err; } if (argc) |