You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by st...@apache.org on 2001/12/18 22:56:16 UTC
cvs commit: httpd-2.0/modules/arch/win32 mod_win32.c
stoddard 01/12/18 13:56:16
Modified: . CHANGES
modules/arch/win32 mod_win32.c
Log:
Win32: Fix bug that could cause CGI scripts with QUERY_STRINGS
to fail.
Gotta allocate enough space for that NULL string terminator :-)
Revision Changes Path
1.479 +2 -0 httpd-2.0/CHANGES
Index: CHANGES
===================================================================
RCS file: /home/cvs/httpd-2.0/CHANGES,v
retrieving revision 1.478
retrieving revision 1.479
diff -u -r1.478 -r1.479
--- CHANGES 2001/12/18 13:48:52 1.478
+++ CHANGES 2001/12/18 21:56:15 1.479
@@ -1,4 +1,6 @@
Changes with Apache 2.0.30-dev
+ *) Win32: Fix bug that could cause CGI scripts with QUERY_STRINGS
+ to fail. [Bill Stoddard]
*) Change core code to allow an MPM to set hard thread/server
limits at startup. [Jeff Trawick]
1.9 +1 -1 httpd-2.0/modules/arch/win32/mod_win32.c
Index: mod_win32.c
===================================================================
RCS file: /home/cvs/httpd-2.0/modules/arch/win32/mod_win32.c,v
retrieving revision 1.8
retrieving revision 1.9
diff -u -r1.8 -r1.9
--- mod_win32.c 2001/11/24 00:17:01 1.8
+++ mod_win32.c 2001/12/18 21:56:16 1.9
@@ -284,7 +284,7 @@
const char *cgiprg, const char *cgiargs)
{
apr_array_header_t *args = apr_array_make(p, 8, sizeof(char*));
- char *d = apr_palloc(p, strlen(interp));
+ char *d = apr_palloc(p, strlen(interp)+1);
const char *ch = interp;
const char **arg;
int prgtaken = 0;