cvs commit: apache/support suexec.c

[Marc Slemko]

marc        97/02/20 17:20:41

  Modified:    src       CHANGES
               support   suexec.c
  Log:
  Don't set the PATH variable in the environment to a local string.
  
  Reviewed by:  Randy Terbush, Roy Fielding
  
  Revision  Changes    Path
  1.177     +1 -0      apache/src/CHANGES
  
  Index: CHANGES
  ===================================================================
  RCS file: /export/home/cvs/apache/src/CHANGES,v
  retrieving revision 1.176
  retrieving revision 1.177
  diff -C3 -r1.176 -r1.177
  *** CHANGES   1997/02/21 00:27:08     1.176
  --- CHANGES   1997/02/21 01:20:39     1.177
  ***************
  *** 63,68 ****
  --- 63,69 ----
         - Make wrapper work for files on NFS filesystem.
         - Fix portability problem of MAXPATHLEN.
         - Fix array overrun problem in clean_env().
  +      - Fix allocation of PATH environment variable
    
      *) Removed extraneous blank line is description of mod_status chars.
         [Kurt Kohler]
  
  
  
  1.18      +1 -1      apache/support/suexec.c
  
  Index: suexec.c
  ===================================================================
  RCS file: /export/home/cvs/apache/support/suexec.c,v
  retrieving revision 1.17
  retrieving revision 1.18
  diff -C3 -r1.17 -r1.18
  *** suexec.c  1997/02/13 14:42:08     1.17
  --- suexec.c  1997/02/21 01:20:40     1.18
  ***************
  *** 203,209 ****
        }
    
        sprintf(pathbuf, "PATH=%s", SAFE_PATH);
  !     cleanenv[cidx] = pathbuf;
        cleanenv[++cidx] = NULL;
            
        environ = cleanenv;
  --- 203,209 ----
        }
    
        sprintf(pathbuf, "PATH=%s", SAFE_PATH);
  !     cleanenv[cidx] = strdup(pathbuf);
        cleanenv[++cidx] = NULL;
            
        environ = cleanenv;