Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 9 Jun 2000 00:07:07 -0500
From:      Dan Nelson <dnelson@emsphone.com>
To:        Kris Kennaway <kris@FreeBSD.ORG>
Cc:        Boris Popov <bp@butya.kz>, John LoVerso <loverso@infolibria.com>, current@FreeBSD.ORG
Subject:   Re: mktemp() patch
Message-ID:  <20000609000707.A16619@dan.emsphone.com>
In-Reply-To: <Pine.BSF.4.21.0006082039030.67602-100000@freefall.freebsd.org>; from "Kris Kennaway" on Thu Jun  8 20:47:36 GMT 2000
References:  <Pine.BSF.4.10.10006090704550.86326-100000@lion.butya.kz> <Pine.BSF.4.21.0006082039030.67602-100000@freefall.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
In the last episode (Jun 08), Kris Kennaway said:
> On Fri, 9 Jun 2000, Boris Popov wrote:
> > 	Count both, nwfs and smbfs, because any program can attempt to
> > create temporary file on these filesystems. File with an invalid
> > file name will be rejected, and this will cost an additional lookup
> > operation(s).
> 
> I'm not sure that weird filesystems are a valid argument against
> mktemp() naming - there are LOTS of UNIX code which assumes UNIX
> namespace conventions, and it's not just mktemp() which is going to
> break on weird filesystems. For example, should we limit all FreeBSD
> file names to 8.3 single-case in case someone wants to run from an
> old-style MSDOS partition?

I still suggest not using symbols at all, since I'd like to be able to
quickly remove tempfiles by hand without worrying if I have to escape #
or ^, etc.  Considering the great jump in randomness between the
orginal and the proposed (65536 -> 916132832 just using [A-Za-z0-9] ),
I'd rather stick with easy-to-read and type tempnames.
 
> Basically, I think the answer is not to use a nwfs or smbfs
> filesystem as your TMPDIR :-)

mktemp() doesn't use TMPDIR; the user gets to pass a template of his
choosing, which could reasonably be just "bobXXXXX.tmp".

-- 
	Dan Nelson
	dnelson@emsphone.com


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20000609000707.A16619>