You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@apr.apache.org by bj...@apache.org on 2002/04/09 05:57:47 UTC

cvs commit: apr-util/buckets apr_buckets_file.c

bjh         02/04/08 20:57:47

  Modified:    buckets  apr_buckets_file.c
  Log:
  Manually created bucket needs bucket allocator field set.
  This fixes a segfault parsing large SSIs on non-mmap systems.
  
  Revision  Changes    Path
  1.67      +1 -0      apr-util/buckets/apr_buckets_file.c
  
  Index: apr_buckets_file.c
  ===================================================================
  RCS file: /home/cvs/apr-util/buckets/apr_buckets_file.c,v
  retrieving revision 1.66
  retrieving revision 1.67
  diff -u -r1.66 -r1.67
  --- apr_buckets_file.c	29 Mar 2002 22:29:19 -0000	1.66
  +++ apr_buckets_file.c	9 Apr 2002 03:57:47 -0000	1.67
  @@ -177,6 +177,7 @@
           b->data   = a;
           b->type   = &apr_bucket_type_file;
           b->free   = apr_bucket_free;
  +        b->list   = e->list;
           APR_BUCKET_INSERT_AFTER(e, b);
       }
       else {
  
  
  

Re: cvs commit: apr-util/buckets apr_buckets_file.c

Posted by Cliff Woolley <jw...@virginia.edu>.
On 9 Apr 2002 bjh@apache.org wrote:

> bjh         02/04/08 20:57:47
>
>   Modified:    buckets  apr_buckets_file.c
>   Log:
>   Manually created bucket needs bucket allocator field set.
>   This fixes a segfault parsing large SSIs on non-mmap systems.

DOH!!!!  Great catch... thanks Brian!  I need to remind myself to compile
without APR_HAS_MMAP once in a while to check fo this kind of thing.  :(

--Cliff

--------------------------------------------------------------
   Cliff Woolley
   cliffwoolley@yahoo.com
   Charlottesville, VA