Pay close attention to core_create_req() ... and note

  that not one other member of the r->vars is initialized
  herein.  Move this initialization elsewhere.

  (If this is the 'default' - it really aught to be the
  zero value, for that matter).

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

diff --git a/server/core.c b/server/core.c
index 150f361e95e65b0e6967e05cbaebdca679ea5c80..e6b9169ac98cec235ac25ce1a2ce880f698d8c03 100644 (file)
--- a/server/core.c
+++ b/server/core.c
@@ -3767,11 +3767,6 @@ static int core_create_req(request_rec *r)
 
     ap_set_module_config(r->request_config, &core_module, req_cfg);
 
-    /* Begin by presuming any module can make its own path_info assumptions,
-     * until some module interjects and changes the value.
-     */
-    r->used_path_info = AP_REQ_DEFAULT_PATH_INFO;
-
     return OK;
 }
 

diff --git a/server/protocol.c b/server/protocol.c
index 4efe8c44a942964ea5a3ccfdc2ace63a0aa9fec4..327adb63b83628fd78822d7f1d86b63078cd636a 100644 (file)
--- a/server/protocol.c
+++ b/server/protocol.c
@@ -869,6 +869,11 @@ request_rec *ap_read_request(conn_rec *conn)
     r->status          = HTTP_REQUEST_TIME_OUT;  /* Until we get a request */
     r->the_request     = NULL;
 
+    /* Begin by presuming any module can make its own path_info assumptions,
+     * until some module interjects and changes the value.
+     */
+    r->used_path_info = AP_REQ_DEFAULT_PATH_INFO;
+
     tmp_bb = apr_brigade_create(r->pool, r->connection->bucket_alloc);
 
     /* Get the request... */

diff --git a/server/request.c b/server/request.c
index c807dd0d6c1c8592ede95d5fbb33d2ef42b76b85..31270eeb3043c57052104fae7cc427ed47c27dd5 100644 (file)
--- a/server/request.c
+++ b/server/request.c
@@ -1531,6 +1531,11 @@ static request_rec *make_sub_request(const request_rec *r,
      */
     ap_run_create_request(rnew);
 
+    /* Begin by presuming any module can make its own path_info assumptions,
+     * until some module interjects and changes the value.
+     */
+    rnew->used_path_info = AP_REQ_DEFAULT_PATH_INFO;
+
     return rnew;
 }