From: "Bill Stoddard" <[EMAIL PROTECTED]>
Sent: Friday, August 17, 2001 12:06 PM


> Have we rolled 2.0.24 yet? I think this was the last showstopper. 
> 
> +1 on committing, push the tag and roll.

Hold up, I'm finished as well, but Jeff Trawick's patches to fix the segfaults
in mod_ssl should probably be pushed as well.  Jeff, if you agree, please patch
and push the tags of anything that you trust to resolve segfaults.

[Segfaults are very nasty to spmt MPMs :-]

> ----- Original Message ----- 
> From: "Jeff Trawick" <[EMAIL PROTECTED]>
> To: <[EMAIL PROTECTED]>
> Sent: Friday, August 17, 2001 11:19 AM
> Subject: [PATCH] mod_include fix
> 
> 
> > In the "2.0.24 tagged" thread, Greg mentioned a problem with
> > mod_include.  This fixes it for me.
> > 
> > We're in find_end_sequence() walking over the directive and the rest
> > of the tag but we stop parsing so we don't walk over too much storage
> > (e.g., mmap) at once).  When we resume later, ctx->directive_length
> > has the value it had when we bailed out, so directive_length is wrong
> > and we hit errors like:
> > 
> > [Fri Aug 17 11:12:06 2001] [error] [client 127.0.0.1] unknown directiv
> > e "include virt" in parsed doc /home/trawick/apacheinst/htdocs/manual/
> > mod/mod_include.html
> > 
> > If anyone else is hitting similar problems, please try this patch.
> > 
> > Index: modules/filters/mod_include.c
> > ===================================================================
> > RCS file: /home/cvspublic/httpd-2.0/modules/filters/mod_include.c,v
> > retrieving revision 1.121
> > diff -u -r1.121 mod_include.c
> > --- modules/filters/mod_include.c 2001/08/11 04:04:12 1.121
> > +++ modules/filters/mod_include.c 2001/08/17 15:09:26
> > @@ -270,6 +270,8 @@
> >          }
> >          while (c < buf + len) {
> >              if (ctx->bytes_parsed >= BYTE_COUNT_THRESHOLD) {
> > +                /* gonna start over parsing the directive next time through */
> > +                ctx->directive_length = 0;
> >                  return dptr;
> >              }
> >  
> > 
> > -- 
> > Jeff Trawick | [EMAIL PROTECTED] | PGP public key at web site:
> >        http://www.geocities.com/SiliconValley/Park/9289/
> >              Born in Roswell... married an alien...
> > 
> 
> 

Reply via email to