httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ben Laurie <...@algroup.co.uk>
Subject Re: PATCH NT: spaces in argv[]
Date Sun, 16 Nov 1997 13:11:08 GMT
Paul Sutton wrote:
> 
> On Fri, 14 Nov 1997, Paul Sutton wrote:
> > On Fri, 14 Nov 1997, Ben Laurie wrote:
> > > Hmmm ... well, we can default the path to something without spaces for
> > > now (I think that isn't too painful), and hope not too many people
> > > change it to something with spaces. The fix isn't that hard, just
> > > totally digusting, and haven't had the time to do it (and I'm cooking
> > > major Chinese this evening, so I won't have the time today :-).
> >
> > I've already sent a patch for this. It results in the lose of memory
> > thought at the moment. If Windows is so braindead we really need to code
> > around it, we'll need to pass the parent's pool into the function which
> > spawns children.
> 
> Ok, here is a slightly better patch. This doesn't waste space. We don't
> care about performance since this function is only called to create child
> processes, while most work is done by creating threads inside in a single
> process. Win32 must have the world's worst implementation of spawn().

The snag here is that this only fixes the launch of standby Apaches, but
surely we need to fix _all_ occurences of spawn? I'll try to do a more
general fix.

Cheers,

Ben.

-- 
Ben Laurie            |Phone: +44 (181) 735 0686|Apache Group member
Freelance Consultant  |Fax:   +44 (181) 735 0689|http://www.apache.org
and Technical Director|Email: ben@algroup.co.uk |Apache-SSL author
A.L. Digital Ltd,     |http://www.algroup.co.uk/Apache-SSL
London, England.      |"Apache: TDG" http://www.ora.com/catalog/apache

Mime
View raw message