You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by tr...@apache.org on 2002/07/10 13:58:37 UTC

cvs commit: httpd-2.0/modules/experimental mod_ext_filter.c

trawick     2002/07/10 04:58:37

  Modified:    .        CHANGES
               modules/experimental mod_ext_filter.c
  Log:
  mod_ext_filter: don't segfault if content-type is not set.
  
  PR:                        10617
  Submitted by:              Arthur P. Smith <ap...@aps.org>
  Reviewed/modified by:      Jeff Trawick
  
  Revision  Changes    Path
  1.861     +3 -0      httpd-2.0/CHANGES
  
  Index: CHANGES
  ===================================================================
  RCS file: /home/cvs/httpd-2.0/CHANGES,v
  retrieving revision 1.860
  retrieving revision 1.861
  diff -u -r1.860 -r1.861
  --- CHANGES	8 Jul 2002 03:40:36 -0000	1.860
  +++ CHANGES	10 Jul 2002 11:58:36 -0000	1.861
  @@ -1,5 +1,8 @@
   Changes with Apache 2.0.40
   
  +  *) mod_ext_filter: don't segfault if content-type is not set.  PR 10617.
  +     [Arthur P. Smith <ap...@aps.org>, Jeff Trawick]
  +
     *) Fix an error when building ApacheMonitor. 
        [James Cox <im...@php.net>]
   
  
  
  
  1.32      +2 -1      httpd-2.0/modules/experimental/mod_ext_filter.c
  
  Index: mod_ext_filter.c
  ===================================================================
  RCS file: /home/cvs/httpd-2.0/modules/experimental/mod_ext_filter.c,v
  retrieving revision 1.31
  retrieving revision 1.32
  diff -u -r1.31 -r1.32
  --- mod_ext_filter.c	28 Jun 2002 08:40:24 -0000	1.31
  +++ mod_ext_filter.c	10 Jul 2002 11:58:37 -0000	1.32
  @@ -517,7 +517,8 @@
       ctx->p = f->r->pool;
       if (ctx->filter->intype &&
           ctx->filter->intype != INTYPE_ALL &&
  -        strcasecmp(ctx->filter->intype, f->r->content_type)) {
  +        (!f->r->content_type ||
  +         strcasecmp(ctx->filter->intype, f->r->content_type))) {
           /* wrong IMT for us; don't mess with the output */
           ctx->noop = 1;
       }