From owner-freebsd-rc@FreeBSD.ORG Wed May 31 18:21:52 2006 Return-Path: X-Original-To: freebsd-rc@freebsd.org Delivered-To: freebsd-rc@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 26FE516B531; Wed, 31 May 2006 18:21:52 +0000 (UTC) (envelope-from brdavis@odin.ac.hmc.edu) Received: from odin.ac.hmc.edu (Odin.AC.HMC.Edu [134.173.32.75]) by mx1.FreeBSD.org (Postfix) with ESMTP id C1EB943D48; Wed, 31 May 2006 18:21:51 +0000 (GMT) (envelope-from brdavis@odin.ac.hmc.edu) Received: from odin.ac.hmc.edu (localhost.localdomain [127.0.0.1]) by odin.ac.hmc.edu (8.13.0/8.13.0) with ESMTP id k4VILogK022402; Wed, 31 May 2006 11:21:50 -0700 Received: (from brdavis@localhost) by odin.ac.hmc.edu (8.13.0/8.13.0/Submit) id k4VILkmS022394; Wed, 31 May 2006 11:21:46 -0700 Date: Wed, 31 May 2006 11:21:46 -0700 From: Brooks Davis To: Florent Thoumie Message-ID: <20060531182146.GC20044@odin.ac.hmc.edu> References: <200605311440.k4VEeJsu051493@freefall.freebsd.org> <1149089131.91276.37.camel@mayday.esat.net> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="Sr1nOIr3CvdE5hEN" Content-Disposition: inline In-Reply-To: <1149089131.91276.37.camel@mayday.esat.net> User-Agent: Mutt/1.4.1i X-Virus-Scanned: by amavisd-new Cc: Christopher Nelson , freebsd-rc@freebsd.org Subject: Re: conf/97301: infinite loop on boot when network_interfaces set to nonexistent interface in rc.conf X-BeenThere: freebsd-rc@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Discussion related to /etc/rc.d design and implementation." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 31 May 2006 18:22:00 -0000 --Sr1nOIr3CvdE5hEN Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, May 31, 2006 at 04:25:31PM +0100, Florent Thoumie wrote: > On Wed, 2006-05-31 at 14:40 +0000, Christopher Nelson wrote: > =20 > > Duh. I knew that. The bug was not submitted because I thought that I > > had the separator correct. The bug was submitted because just having = an > > incorrect separator should NOT cause the entire system to get stuck in > > an infinite loop on startup. That behavior is WAY too pathological for > > such a minor mistake. >=20 > That wasn't really clear since you talked about non-existent > interfaces. /etc/rc.d/netif is perfectly happy with them. >=20 > We could 's/[^a-z0-9 ]/ /g' but sed isn't available at that time and I > don't feel like maintaining a list of wrong separators to use with > ltr(). IMO we should make two changes here. First, if network_interfaces is not set to "auto" we should issue a warning when rc.d/netif is run indicating that doing so is deprecated. Second, in the list_net_interfaces function in network.subr we should verify that each entry is valid and fail with an error message and "exit 1" if it isn't. That will cause the system to not boot when misconfigured, but that's OK with me since I think we should kill off this variable entirely and thus I don't think it's worth any more effort to handle errors while we wait for a good time to finish it off. The only reason I haven't killed in in HEAD is that I haven't figured out how to refactor rc.conf(5) without the network_interfaces variable to hang all the configuration stuff off of. -- Brooks --=20 Any statement of the form "X is the one, true Y" is FALSE. PGP fingerprint 655D 519C 26A7 82E7 2529 9BF0 5D8E 8BE9 F238 1AD4 --Sr1nOIr3CvdE5hEN Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.1 (GNU/Linux) iD8DBQFEfd66XY6L6fI4GtQRAhKoAJ9U/dolM1Evx0yRybbGTyfhUGarkACfe80o vgV0WnTFDJ2J9Hhs9PupbYA= =0Rap -----END PGP SIGNATURE----- --Sr1nOIr3CvdE5hEN--