Stop unsetting the 'empty' query string result with

  a NULL argument in ecb->lpszQueryString, eliminating segfaults
  for some ISAPI modules.

PR: 14399
Submitted by:	Detlev Vendt <detlev.vendt@brillit.de>

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@97468 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/modules/arch/win32/mod_isapi.c b/modules/arch/win32/mod_isapi.c
index db13b4d64b5d61ff244435b7a512b0a6843b12c3..83032ab3387343f03a3caa6e574f9c9d8f46c35d 100644 (file)
--- a/modules/arch/win32/mod_isapi.c
+++ b/modules/arch/win32/mod_isapi.c
@@ -1422,12 +1422,6 @@ apr_status_t isapi_handler (request_rec *r)
     cid->ecb->lpszPathTranslated = (char*) apr_table_get(e, "PATH_TRANSLATED");
     cid->ecb->lpszContentType = (char*) apr_table_get(e, "CONTENT_TYPE");
     
-    /* Based on some examples I've noticed, NULL is expected here. 
-     */
-    if (!*cid->ecb->lpszQueryString) {
-        cid->ecb->lpszQueryString = NULL;
-    }
-
     /* Set up the callbacks */
     cid->ecb->GetServerVariable = GetServerVariable;
     cid->ecb->WriteClient = WriteClient;