CVS commit: src/libexec/httpd

[Thor Lancelot Simon <tls%netbsd.org@localhost>]

Module Name:    src
Committed By:   tls
Date:           Thu Feb 19 22:33:39 UTC 2009

Modified Files:
        src/libexec/httpd: cgi-bozo.c

Log Message:
More CGI handling fixes from Sergey Katsev at Coyote Point.

1) Fix an issue where because of the reordering of transform_request
   and process_cgi, the cgi-script name was being cut off
   by one character (transform_request for some reason
   cuts off the leading '/' for the file name as part of its
   processing).  As an 'easy' fix, simply re-add the '/' to the
   front of the filename, which means that we don't have to
   mess with the logic that sometimes adds +1 and sometimes doesn't.

2) Work around ridiculous bug in PHP reported by lukem in 2004,
   but stubbornly never fixed by the PHP maintainers:

   Change the SCRIPT_NAME and SCRIPT_FILENAME variables
   to contain the file name only, not the query if one exists.
   Having the query in SCRIPT_FILENAME causes php-cgi to not
   work, as per the bug information here:
         http://bugs.php.net/bug.php?id=28227

3) Fix a memory leak because URL wasn't being freed.


To generate a diff of this commit:
cvs rdiff -r1.9 -r1.10 src/libexec/httpd/cgi-bozo.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.