| Message ID | 20201028115349.6855-1-lukas.bulwahn@gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | ipv6: mcast: make annotations for ip6_mc_msfget() consistent | expand |
On Wed, 28 Oct 2020 12:53:49 +0100 Lukas Bulwahn wrote: > Commit 931ca7ab7fe8 ("ip*_mc_gsfget(): lift copyout of struct group_filter > into callers") adjusted the type annotations for ip6_mc_msfget() at its > declaration, but missed the type annotations at its definition. > > Hence, sparse complains on ./net/ipv6/mcast.c: > > mcast.c:550:5: error: symbol 'ip6_mc_msfget' redeclared with different type \ > (incompatible argument 3 (different address spaces)) > > Make ip6_mc_msfget() annotations consistent, which also resolves this > warning from sparse: > > mcast.c:607:34: warning: incorrect type in argument 1 (different address spaces) > mcast.c:607:34: expected void [noderef] __user *to > mcast.c:607:34: got struct __kernel_sockaddr_storage *p > > No functional change. No change in object code. > > Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com> Applied, thank you!
diff --git a/net/ipv6/mcast.c b/net/ipv6/mcast.c index 8cd2782a31e4..6c8604390266 100644 --- a/net/ipv6/mcast.c +++ b/net/ipv6/mcast.c @@ -548,7 +548,7 @@ int ip6_mc_msfilter(struct sock *sk, struct group_filter *gsf, } int ip6_mc_msfget(struct sock *sk, struct group_filter *gsf, - struct sockaddr_storage *p) + struct sockaddr_storage __user *p) { int err, i, count, copycount; const struct in6_addr *group;
Commit 931ca7ab7fe8 ("ip*_mc_gsfget(): lift copyout of struct group_filter into callers") adjusted the type annotations for ip6_mc_msfget() at its declaration, but missed the type annotations at its definition. Hence, sparse complains on ./net/ipv6/mcast.c: mcast.c:550:5: error: symbol 'ip6_mc_msfget' redeclared with different type \ (incompatible argument 3 (different address spaces)) Make ip6_mc_msfget() annotations consistent, which also resolves this warning from sparse: mcast.c:607:34: warning: incorrect type in argument 1 (different address spaces) mcast.c:607:34: expected void [noderef] __user *to mcast.c:607:34: got struct __kernel_sockaddr_storage *p No functional change. No change in object code. Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com> --- applies cleanly on current master and next-20201028 David, Jakub, please pick this minor non-urgent clean-up patch. net/ipv6/mcast.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)