There is a problem with assigning 127.0.0.1 here. Revert the call

to in_up_loopback(). There is an issue when doing ifconfig lo0 127.0.0.1/8
the system ends up with 2 127.0.0.1 routes which breaks things like smptd.
Found by tb@

diff --git a/sys/net/if.c b/sys/net/if.c
index 94df391..ffafd37 100644 (file)
--- a/sys/net/if.c
+++ b/sys/net/if.c
@@ -1,4 +1,4 @@
-/*     $OpenBSD: if.c,v 1.543 2018/02/10 05:52:08 florian Exp $        */
+/*     $OpenBSD: if.c,v 1.544 2018/02/10 09:32:54 claudio Exp $        */
 /*     $NetBSD: if.c,v 1.35 1996/05/07 05:26:04 thorpej Exp $  */
 
 /*
@@ -1552,7 +1552,6 @@ if_up(struct ifnet *ifp)
        /* Userland expects the kernel to set ::1 on default lo(4). */
        if (ifp->if_index == rtable_loindex(ifp->if_rdomain)) {
                in6_ifattach(ifp);
-               in_up_loopback(ifp);
        }
 #endif