Re: [PATCH bpf] bpf: fix the check that forwarding is enabled in bpf_ipv6_fib_lookup

2019-06-17 Thread Daniel Borkmann
On 06/16/2019 12:53 AM, Anton Protopopov wrote: > The bpf_ipv6_fib_lookup function should return BPF_FIB_LKUP_RET_FWD_DISABLED > when forwarding is disabled for the input device. However instead of checking > if forwarding is enabled on the input device, it checked the global >

Re: [PATCH bpf] bpf: fix the check that forwarding is enabled in bpf_ipv6_fib_lookup

2019-06-17 Thread David Ahern
On 6/17/19 5:17 AM, Toke Høiland-Jørgensen wrote: > Anton Protopopov writes: > >> The bpf_ipv6_fib_lookup function should return BPF_FIB_LKUP_RET_FWD_DISABLED >> when forwarding is disabled for the input device. However instead of >> checking >> if forwarding is enabled on the input device, it

Re: [PATCH bpf] bpf: fix the check that forwarding is enabled in bpf_ipv6_fib_lookup

2019-06-17 Thread Toke Høiland-Jørgensen
Anton Protopopov writes: > The bpf_ipv6_fib_lookup function should return BPF_FIB_LKUP_RET_FWD_DISABLED > when forwarding is disabled for the input device. However instead of checking > if forwarding is enabled on the input device, it checked the global > net->ipv6.devconf_all->forwarding flag.

[PATCH bpf] bpf: fix the check that forwarding is enabled in bpf_ipv6_fib_lookup

2019-06-15 Thread Anton Protopopov
The bpf_ipv6_fib_lookup function should return BPF_FIB_LKUP_RET_FWD_DISABLED when forwarding is disabled for the input device. However instead of checking if forwarding is enabled on the input device, it checked the global net->ipv6.devconf_all->forwarding flag. Change it to behave as expected.