Message ID | 20180719075659.9800-1-yuehaibing@huawei.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
Series | [v2,bpf-next] bpfilter: Fix mismatch in function argument types | expand |
On Thu, Jul 19, 2018 at 03:56:59PM +0800, YueHaibing wrote: > Fix following warning: > net/ipv4/bpfilter/sockopt.c:28:5: error: symbol 'bpfilter_ip_set_sockopt' redeclared with different type > net/ipv4/bpfilter/sockopt.c:34:5: error: symbol 'bpfilter_ip_get_sockopt' redeclared with different type > > Signed-off-by: YueHaibing <yuehaibing@huawei.com> LGTM. Missing a Fixes tag though. Fixes: d2ba09c17a06 ("net: add skeleton of bpfilter kernel module") Acked-by: Martin KaFai Lau <kafai@fb.com>
On 07/19/2018 05:48 PM, Martin KaFai Lau wrote: > On Thu, Jul 19, 2018 at 03:56:59PM +0800, YueHaibing wrote: >> Fix following warning: >> net/ipv4/bpfilter/sockopt.c:28:5: error: symbol 'bpfilter_ip_set_sockopt' redeclared with different type >> net/ipv4/bpfilter/sockopt.c:34:5: error: symbol 'bpfilter_ip_get_sockopt' redeclared with different type >> >> Signed-off-by: YueHaibing <yuehaibing@huawei.com> > LGTM. Missing a Fixes tag though. > > Fixes: d2ba09c17a06 ("net: add skeleton of bpfilter kernel module") > Acked-by: Martin KaFai Lau <kafai@fb.com> (Should go to -net tree.) Acked-by: Daniel Borkmann <daniel@iogearbox.net>
From: Daniel Borkmann <daniel@iogearbox.net> Date: Thu, 19 Jul 2018 17:51:41 +0200 > On 07/19/2018 05:48 PM, Martin KaFai Lau wrote: >> On Thu, Jul 19, 2018 at 03:56:59PM +0800, YueHaibing wrote: >>> Fix following warning: >>> net/ipv4/bpfilter/sockopt.c:28:5: error: symbol 'bpfilter_ip_set_sockopt' redeclared with different type >>> net/ipv4/bpfilter/sockopt.c:34:5: error: symbol 'bpfilter_ip_get_sockopt' redeclared with different type >>> >>> Signed-off-by: YueHaibing <yuehaibing@huawei.com> >> LGTM. Missing a Fixes tag though. >> >> Fixes: d2ba09c17a06 ("net: add skeleton of bpfilter kernel module") >> Acked-by: Martin KaFai Lau <kafai@fb.com> > > (Should go to -net tree.) > > Acked-by: Daniel Borkmann <daniel@iogearbox.net> Applied.
diff --git a/include/linux/bpfilter.h b/include/linux/bpfilter.h index 687b176..f02cee0 100644 --- a/include/linux/bpfilter.h +++ b/include/linux/bpfilter.h @@ -5,10 +5,10 @@ #include <uapi/linux/bpfilter.h> struct sock; -int bpfilter_ip_set_sockopt(struct sock *sk, int optname, char *optval, +int bpfilter_ip_set_sockopt(struct sock *sk, int optname, char __user *optval, unsigned int optlen); -int bpfilter_ip_get_sockopt(struct sock *sk, int optname, char *optval, - int *optlen); +int bpfilter_ip_get_sockopt(struct sock *sk, int optname, char __user *optval, + int __user *optlen); extern int (*bpfilter_process_sockopt)(struct sock *sk, int optname, char __user *optval, unsigned int optlen, bool is_set);
Fix following warning: net/ipv4/bpfilter/sockopt.c:28:5: error: symbol 'bpfilter_ip_set_sockopt' redeclared with different type net/ipv4/bpfilter/sockopt.c:34:5: error: symbol 'bpfilter_ip_get_sockopt' redeclared with different type Signed-off-by: YueHaibing <yuehaibing@huawei.com> --- include/linux/bpfilter.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)