From owner-freebsd-bugs@FreeBSD.ORG Tue Aug 17 18:51:50 2010 Return-Path: Delivered-To: freebsd-bugs@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 7B26B1065695; Tue, 17 Aug 2010 18:51:50 +0000 (UTC) (envelope-from walter.pelissero@iesy.net) Received: from mail01.ish.de (mailout.ish.de [80.69.98.247]) by mx1.freebsd.org (Postfix) with ESMTP id AC0CA8FC27; Tue, 17 Aug 2010 18:51:48 +0000 (UTC) Received: from [95.222.206.96] (account walter.pelissero@iesy.net HELO zaphod.home.lan) by mail-fe-02.mail01.ish.de (CommuniGate Pro SMTP 5.2.18) with ESMTPSA id 374816000; Tue, 17 Aug 2010 20:51:47 +0200 Received: from zaphod.home.lan (localhost [127.0.0.1]) by zaphod.home.lan (8.14.4/8.14.4) with ESMTP id o7HInVaU049244; Tue, 17 Aug 2010 20:49:31 +0200 (CEST) (envelope-from wcp@zaphod.home.lan) Received: (from wcp@localhost) by zaphod.home.lan (8.14.4/8.14.4/Submit) id o7HInVZM049243; Tue, 17 Aug 2010 20:49:31 +0200 (CEST) (envelope-from wcp) From: walter@pelissero.de (Walter C. Pelissero) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Message-ID: <19562.55739.75948.533618@zaphod.home.lan> Date: Tue, 17 Aug 2010 20:49:31 +0200 To: Oliver Fromme In-Reply-To: <201008171506.o7HF6QL7075437@lurza.secnetix.de> References: <19562.39686.747733.126631@zaphod.home.lan> <201008171506.o7HF6QL7075437@lurza.secnetix.de> X-Mailer: VM 8.1.1 under 23.2.1 (i386-portbld-freebsd8.1) X-Attribution: WP X-For-Spammers: blacklistme@pelissero.de X-MArch-Archive-ID: 129377 X-MArch-Processing-Time: 0.37s Cc: freebsd-bugs@FreeBSD.ORG, bug-followup@FreeBSD.ORG Subject: Re: bin/149424: fstab and labels with whitespace X-BeenThere: freebsd-bugs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: walter@pelissero.de List-Id: Bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 17 Aug 2010 18:51:50 -0000 Oliver Fromme writes: > Walter C. Pelissero wrote: > > Oliver Fromme writes: > > > Sure, but still everyone who uses backslashes will have to > > > change his /etc/fstab *in advance*, or otherwise his box > > > will not boot after the update. > > > > It seems to be unlikely there are systems out there relying on a > > boot-time mounted filesystem whose device path (or label) contains a > > backslash. Certainly not a great deal of those. > > > > Maybe you care to give us some concrete example. Who knows, there > > could be a BSD distribution, that I failed to notice, with a really > > bizarre naming convention for the device drivers. > > What prevents you from labelling a disk 'backup\500GB'? Besides some remorse about the uninspired name, nothing keeps you from that. Are you also going to boot from that volume? Because that is what you were arguing just before. So, if you aren't, your argument about rendering systems unbootable is moot. BTW, that was the meaning of: It seems to be unlikely there are systems out there relying on a boot-time mounted filesystem whose device path (or label) contains a backslash. Certainly not a great deal of those." > In the past 16 years (maybe longer; I checked only the FreeBSD > repository) nobody dared to change the syntax of /etc/fstab. I'm > not going to open this can of worms. Are you the person responsible for that code? > > > > > in /etc/fstab, for whatever reason. This change would > > > > > make them rather unhappy, I'm afraid. > > > > > > > > Being unhappy for having to change a couple of entries in fstab is not > > > > nearly as bad as not being able to insert an entry altogether. > > > > > > As I wrote, there are several workarounds. > > > > Then, why do you write it again? > > Because it seemed that you missed it, Did I? > Breaking existing configurations (possibly leaving people > with unbootable remote machines!) Right: machines that boot from backup\500GB, aren't they? > is certainly worse than requiring some people to use workarounds > for an optional feature. Booting from backup\500GB sounds pretty "optional feature" to me. > And finally, the patch presented in this PR fails to update all > places that handle /etc/fstab. For example, it breaks > /etc/rc.d/gbde, /etc/rc.d/jail and "mount -p". Neither /etc/rc.d/gbde nor /etc/rc.d/jail is broken by my patch. (As far as I can tell, there is not even use for the device name in /etc/rc.d/jail.) It would, on the other hand, interfere with your ill named USB disk if that were encrypted with GBDE (a still experimental feature). > Those are just three things from the top of my head; there are > probably more. There must be aplenty, if in 8 years nobody hasn't been able to even enumerate them. > Not to mention any third-party software that might try to parse > that file. Parsing is not hindered by my patch. Programs that use libc to read fstab don't have problems. Those that use some home brewed parser are still able to parse fstab without problems, because the syntax is backward compatible. They simply don't get the right value for those device entries that employ backslashes. In other words, you are in trouble if you intend to use some\funny\device in your third-party software that doesn't use libc to parse fstab. > If you want to change the syntax of /etc/fstab, *all* of the above > needs to be adapted. That patch changes libc. Where else more central or appropriate did you have in mind? /sbin/mount, for -p to work, would require an equally trivial patch. So would some other program, I suppose. Hardly a tin of invertebrates, and nothing that cannot be discovered by trial-and- error. If you meant to say: "either you mend all FreeBSD boot chain or your patch is not enough"; well, I'm still waiting to see your code. Or the others of your "several" workarounds. I've seen one this far. (Yes, I didn't miss it.) -- walter pelissero http://www.pelissero.de