ipv6/route: return if there is no fib_nh_gw_family
authorHangbin Liu <liuhangbin@gmail.com>
Wed, 20 Nov 2019 07:39:06 +0000 (15:39 +0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 20 Nov 2019 20:17:39 +0000 (12:17 -0800)
Previously we will return directly if (!rt || !rt->fib6_nh.fib_nh_gw_family)
in function rt6_probe(), but after commit cc3a86c802f0
("ipv6: Change rt6_probe to take a fib6_nh"), the logic changed to
return if there is fib_nh_gw_family.

Fixes: cc3a86c802f0 ("ipv6: Change rt6_probe to take a fib6_nh")
Signed-off-by: Hangbin Liu <liuhangbin@gmail.com>
Reviewed-by: David Ahern <dsahern@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv6/route.c

index e60bf8e..3f83ea8 100644 (file)
@@ -634,7 +634,7 @@ static void rt6_probe(struct fib6_nh *fib6_nh)
         * Router Reachability Probe MUST be rate-limited
         * to no more than one per minute.
         */
-       if (fib6_nh->fib_nh_gw_family)
+       if (!fib6_nh->fib_nh_gw_family)
                return;
 
        nh_gw = &fib6_nh->fib_nh_gw6;