On Tue, 16 Sep 2008, Simon Horman wrote:
> ip_vs_conn_new expects a union nf_inet_addr as the type for its address
> parameters, not a plain integer.
>
> This problem was detected by sparse.
>
> make C=1
> CHECK net/ipv4/ipvs/ip_vs_core.c
> net/ipv4/ipvs/ip_vs_core.c:469:9: warning: Using plain integer as NULL pointer
>
> Signed-off-by: Simon Horman <horms@xxxxxxxxxxxx>
>
> Index: lvs-2.6/net/ipv4/ipvs/ip_vs_core.c
> ===================================================================
> --- lvs-2.6.orig/net/ipv4/ipvs/ip_vs_core.c 2008-09-16 15:28:40.000000000 +1000
> +++ lvs-2.6/net/ipv4/ipvs/ip_vs_core.c 2008-09-16 15:28:47.000000000 +1000
> @@ -457,6 +457,7 @@ int ip_vs_leave(struct ip_vs_service *sv
> if (sysctl_ip_vs_cache_bypass && svc->fwmark && unicast) {
> int ret, cs;
> struct ip_vs_conn *cp;
> + union nf_inet_addr daddr = { .all = { 0, 0, 0, 0 } };
Could be static and const, to avoid allocation and initialization on the
stack every time. But I don't think this a critical code path.
>
> ip_vs_service_put(svc);
>
> @@ -465,7 +466,7 @@ int ip_vs_leave(struct ip_vs_service *sv
> cp = ip_vs_conn_new(svc->af, iph.protocol,
> &iph.saddr, pptr[0],
> &iph.daddr, pptr[1],
> - 0, 0,
> + &daddr, 0,
> IP_VS_CONN_F_BYPASS,
> NULL);
> if (cp == NULL)
--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
