Mailing-List: contact new-httpd-help@apache.org; run by ezmlm
Precedence: bulk
Reply-To: new-httpd@apache.org
Sender: jfclere@vtxrm2.bcn.fsc.net
Message-ID: <3AE81D0C.244D4732@fujitsu-siemens.com>
Date: Thu, 26 Apr 2001 15:05:16 +0200
From: jean-frederic clere <jfrederic.clere@fujitsu-siemens.com>
MIME-Version: 1.0
To: new-httpd@apache.org
Subject: Re: [STATUS] (apache-1.3) Wed Apr 25 23:45:04 EDT 2001
References: <Pine.LNX.4.30.0104252345140.3972-100000@mail.gator.net>
 <m3zod3rk4b.fsf@adsl-77-241-65.rdu.bellsouth.net>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit

Jeff Trawick wrote:
> 
> Blue Lang <blue@gator.net> writes:
> 
> > On Wed, 25 Apr 2001, Rodent of Unusual Size wrote:
> >
> > > RELEASE NON-SHOWSTOPPERS BUT WOULD BE REAL NICE TO WRAP THESE UP:
> > >
> > >     * htpasswd.c and htdigest.c use tmpnam()... consider using
> > >       mkstemp() when available.
> > >         Message-ID: <Pine.LNX.4.30.0102191441100.22500-100000@dax.joh.cam.ac.uk>
> > >     Status:
> >
> > there have been a couple of patches out there to deal with this.. were
> > there problems with them? will they only be accepted if they work on 1.3
> > and 2.0?
> 
> IMHO, for 2.0 what we need is an APR function (or a special
> apr_file_open() flag) which creates a temporary file according to the
> needs of the platform (probably it uses mkstemp() on Unix) and returns
> an APR representation of the file.

There is a mktemp.c (for apr_mkstemp()) in APR, well it need more documentations
and may be testing ;=).

> 
> This would fit in well with htdigest which already uses APR file I/O.
> htpasswd uses stdio, for better or for worse.
> 
> I have no strong opinion for 1.3, but: I don't think mkstemp exists on
> all supported platforms, the use of mkstemp()-created files with stdio
> may not be portable across all supported platforms, and there is no
> APR to handle any nuances, so I suspect the 1.3 apps would get a
> little uglier and a little less stable (i.e., might break on some
> platforms).
> --
> Jeff Trawick | trawickj@bellsouth.net | PGP public key at web site:
>        http://www.geocities.com/SiliconValley/Park/9289/
>              Born in Roswell... married an alien...