Remove a bad typo introduced in rev 1.185 and found by jsg@.
authormpi <mpi@openbsd.org>
Mon, 20 Apr 2015 09:12:57 +0000 (09:12 +0000)
committermpi <mpi@openbsd.org>
Mon, 20 Apr 2015 09:12:57 +0000 (09:12 +0000)
Because of this semicolon, a local route was *always* created.  This
should have no impact except for IPv6 addresses on loopback interfaces
for which the IPv6 code was trying to create a route to "::1" twice.

This should now be fixed so we can fix the bug that was hidding a bug
who was hidding a bug... lalala...

ok henning@

sys/net/route.c

index 47824fd..a3d660a 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: route.c,v 1.208 2015/03/26 11:02:44 mpi Exp $ */
+/*     $OpenBSD: route.c,v 1.209 2015/04/20 09:12:57 mpi Exp $ */
 /*     $NetBSD: route.c,v 1.14 1996/02/13 22:00:46 christos Exp $      */
 
 /*
@@ -1268,7 +1268,7 @@ rt_ifa_addlocal(struct ifaddr *ifa)
 
        /* If there is no loopback entry, allocate one. */
        rt = rtalloc(ifa->ifa_addr, 0, ifa->ifa_ifp->if_rdomain);
-       if (rt == NULL || !ISSET(rt->rt_flags, flags));
+       if (rt == NULL || !ISSET(rt->rt_flags, flags))
                error = rt_ifa_add(ifa, RTF_UP | flags, ifa->ifa_addr);
        if (rt)
                rtfree(rt);