From: Daniel Borkmann <dan...@iogearbox.net>
Date: Tue, 8 Sep 2015 18:00:09 +0200
> 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: db20fd2b0108 ("bpf: add lookup/update/delete/iterate methods to BPF
> maps")
> Signed-off-by: Daniel Borkmann <dan...@iogearbox.net>
> Acked-by: Alexei Starovoitov <a...@plumgrid.com>
Applied, thanks Daniel.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
