Message ID | 3289bb18e08a3d2ab5828c2c5fca9469c505af24.1550249383.git.aclaudi@redhat.com |
---|---|
State | Accepted |
Delegated to: | Pablo Neira |
Headers | show |
Series | [nf] ipvs: fix warning on unused variable | expand |
On Fri, Feb 15, 2019 at 05:51:48PM +0100, Andrea Claudi wrote: > When CONFIG_IP_VS_IPV6 is not defined, build produced this warning: > > net/netfilter/ipvs/ip_vs_ctl.c:899:6: warning: unused variable ‘ret’ [-Wunused-variable] > int ret = 0; > ^~~ > > Fix this by moving the declaration of 'ret' in the CONFIG_IP_VS_IPV6 > section in the same function. > > While at it, drop its unneeded initialisation. This is a follow up to amend the previous patch, I'm going to apply this and pass it to David, thanks.
diff --git a/net/netfilter/ipvs/ip_vs_ctl.c b/net/netfilter/ipvs/ip_vs_ctl.c index 86afacb07e5f..ac8d848d7624 100644 --- a/net/netfilter/ipvs/ip_vs_ctl.c +++ b/net/netfilter/ipvs/ip_vs_ctl.c @@ -896,12 +896,13 @@ ip_vs_new_dest(struct ip_vs_service *svc, struct ip_vs_dest_user_kern *udest, { struct ip_vs_dest *dest; unsigned int atype, i; - int ret = 0; EnterFunction(2); #ifdef CONFIG_IP_VS_IPV6 if (udest->af == AF_INET6) { + int ret; + atype = ipv6_addr_type(&udest->addr.in6); if ((!(atype & IPV6_ADDR_UNICAST) || atype & IPV6_ADDR_LINKLOCAL) &&