summaryrefslogtreecommitdiffstats
path: root/sys/net/if_spppsubr.c
diff options
context:
space:
mode:
authorjoerg <joerg@FreeBSD.org>2001-12-26 20:28:41 +0000
committerjoerg <joerg@FreeBSD.org>2001-12-26 20:28:41 +0000
commiteb6c6922fca06dfd53e72a1c466f4d42fe6cb3a6 (patch)
tree970b558a521f5bde70c5193f2c8608d2aabc6077 /sys/net/if_spppsubr.c
parent48a2f127d42ec6c0c23242a73a5d78ecc5ac0994 (diff)
downloadFreeBSD-src-eb6c6922fca06dfd53e72a1c466f4d42fe6cb3a6.zip
FreeBSD-src-eb6c6922fca06dfd53e72a1c466f4d42fe6cb3a6.tar.gz
For SIOCSIFADDR, don't call if_up() since it would attempt to add the
route to the destination twice. Now that brian has fixed route.c to no longer accept this second route, this long-standing nuisance became a showstopper bug for sppp users. In retrospect, this is the same fix as the one in rev 1.78 of if_sl.c; most likely the original version of sppp has been cloned from SLIP. ;-)
Diffstat (limited to 'sys/net/if_spppsubr.c')
-rw-r--r--sys/net/if_spppsubr.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/sys/net/if_spppsubr.c b/sys/net/if_spppsubr.c
index 7c8cbc5..0fcb0e5 100644
--- a/sys/net/if_spppsubr.c
+++ b/sys/net/if_spppsubr.c
@@ -1024,7 +1024,8 @@ sppp_ioctl(struct ifnet *ifp, IOCTL_CMD_T cmd, void *data)
break;
case SIOCSIFADDR:
- if_up(ifp);
+ /* set the interface "up" when assigning an IP address */
+ ifp->if_flags |= IFF_UP;
/* fall through... */
case SIOCSIFFLAGS:
OpenPOWER on IntegriCloud