]> git.baikalelectronics.ru Git - kernel.git/commit
ebpf: fix fd refcount leaks related to maps in bpf syscall
authorDaniel Borkmann <daniel@iogearbox.net>
Tue, 8 Sep 2015 16:00:09 +0000 (18:00 +0200)
committerDavid S. Miller <davem@davemloft.net>
Wed, 9 Sep 2015 19:39:34 +0000 (12:39 -0700)
commitb076108fc02dc8fb9a2f4be94b4872b7e114966c
treefe34e9afb9076fab7d81b95613b96436e547fa50
parent40709376953a39a17405129ac57574eb2f9e32ef
ebpf: fix fd refcount leaks related to maps in bpf syscall

We may already have gotten a proper fd struct through fdget(), so
whenever we return at the end of an map operation, we need to call
fdput(). However, each map operation from syscall side first probes
CHECK_ATTR() to verify that unused fields in the bpf_attr union are
zero.

In case of malformed input, we return with error, but the lookup to
the map_fd was already performed at that time, so that we return
without an corresponding fdput(). Fix it by performing an fdget()
only right before bpf_map_get(). The fdget() invocation on maps in
the verifier is not affected.

Fixes: 63f52b298909 ("bpf: add lookup/update/delete/iterate methods to BPF maps")
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: Alexei Starovoitov <ast@plumgrid.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
kernel/bpf/syscall.c