You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by rb...@covalent.net on 2000/07/06 16:59:04 UTC

Re: [PATCH] make mod_cgid.c compile

What is the problem you are seeing?  The current tree compiles just fine
for me, and it makes sense.  The socket is allocated out of the pcgi pool,
so we allocate the iol out of the same pool.  To allocate the iol out of a
different pool than the socket itself means that potentially we have the
iol sitting around after the underlying socket is no longer valid.

Ryan

On Thu, 6 Jul 2000, Ed Korthof wrote:

> the head of the anoncvs tree has a problem w/ mod_cgid.c ... the attached
> patch makes it compile.  it's lightly tested (if someone else is working
> on this, please ignore it; i'm just posting it 'cause i wanted to test
> something else out and this was easy to fix).
> 
> thanks --
> 
> ed
> 
> 
> 
> 


_______________________________________________________________________________
Ryan Bloom                        	rbb@apache.org
406 29th St.
San Francisco, CA 94131
-------------------------------------------------------------------------------


Re: [PATCH] make mod_cgid.c compile

Posted by Ed Korthof <ed...@apache.org>.
sorry, i didn't look closely enough at the patch -- it didn't offer
anything against the head of the tree, clearly.  (i started with a
slightly older version of the tree, where it was needed.)  next time, most
sleep before i submit a patch ...

ed

On Thu, 6 Jul 2000 rbb@covalent.net wrote:

> What is the problem you are seeing?  The current tree compiles just fine
> for me, and it makes sense.  The socket is allocated out of the pcgi pool,
> so we allocate the iol out of the same pool.  To allocate the iol out of a
> different pool than the socket itself means that potentially we have the
> iol sitting around after the underlying socket is no longer valid.
> 
> Ryan
> 
> On Thu, 6 Jul 2000, Ed Korthof wrote:
> 
> > the head of the anoncvs tree has a problem w/ mod_cgid.c ... the attached
> > patch makes it compile.  it's lightly tested (if someone else is working
> > on this, please ignore it; i'm just posting it 'cause i wanted to test
> > something else out and this was easy to fix).
> > 
> > thanks --
> > 
> > ed
> > 
> > 
> > 
> > 
> 
> 
> _______________________________________________________________________________
> Ryan Bloom                        	rbb@apache.org
> 406 29th St.
> San Francisco, CA 94131
> -------------------------------------------------------------------------------
>