diff options
| author |   Quentin Monnet <quentin.monnet@netronome.com> | 2019-08-15 15:22:23 +0100 |
|---|---|---|
| committer |   Alexei Starovoitov <ast@kernel.org> | 2019-08-15 22:09:41 -0700 |
| commit | d34b044038bfb0e19caa8b019910efc465f41d5f (patch) | |
| tree | 6972181b15474d393f514cfdd4a8b24d105201e7 /tools/bpf/bpftool/prog.c | |
| parent | selftests/bpf: fix "bind{4, 6} deny specific IP & port" on s390 (diff) | |
| download | linux-dev-d34b044038bfb0e19caa8b019910efc465f41d5f.tar.xz linux-dev-d34b044038bfb0e19caa8b019910efc465f41d5f.zip | |
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/bpf/bpftool/prog.c')
| -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) |