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 2003/11/21 04:49:12 UTC

cvs commit: httpd-2.0/modules/metadata mod_env.c

trawick     2003/11/20 19:49:12

  Modified:    .        CHANGES
               modules/metadata mod_env.c
  Log:
  Fix a problem with the display of empty variables ("SetEnv foo") in
  mod_include.  Now it works like 1.3 (see PR for details).  Setting
  of envvars for scripts was/is consistent with 1.3 for empty variables.
  
  PR:           24734
  Submitted by: Markus Julen <mj zermatt.net>
  Reviewed by:  Jeff Trawick
  
  Revision  Changes    Path
  1.1319    +4 -0      httpd-2.0/CHANGES
  
  Index: CHANGES
  ===================================================================
  RCS file: /home/cvs/httpd-2.0/CHANGES,v
  retrieving revision 1.1318
  retrieving revision 1.1319
  diff -u -r1.1318 -r1.1319
  --- CHANGES	20 Nov 2003 19:53:14 -0000	1.1318
  +++ CHANGES	21 Nov 2003 03:49:12 -0000	1.1319
  @@ -1,6 +1,10 @@
   Changes with Apache 2.1.0-dev
   
     [Remove entries to the current 2.0 section below, when backported]
  +
  +  *) Fix a problem with the display of empty variables ("SetEnv foo") in
  +     mod_include.  PR 24734  [Markus Julen <mj zermatt.net>]
  +
     *) Win32 MPM: The bucket brigades subsystem now honors the MaxMemFree setting.
        [Bill Stoddard]
   
  
  
  
  1.31      +1 -1      httpd-2.0/modules/metadata/mod_env.c
  
  Index: mod_env.c
  ===================================================================
  RCS file: /home/cvs/httpd-2.0/modules/metadata/mod_env.c,v
  retrieving revision 1.30
  retrieving revision 1.31
  diff -u -r1.30 -r1.31
  --- mod_env.c	3 Feb 2003 17:53:09 -0000	1.30
  +++ mod_env.c	21 Nov 2003 03:49:12 -0000	1.31
  @@ -158,7 +158,7 @@
       /* name is mandatory, value is optional.  no value means
        * set the variable to an empty string
        */
  -    apr_table_setn(sconf->vars, name, value);
  +    apr_table_setn(sconf->vars, name, value ? value : "");
   
       return NULL;
   }