Merge r1719016 from trunk:

There is no need to duplicate some memory  here, the 'path' given to ap_parse_htaccess is never modified.
Submitted by: jailletc36
Reviewed/backported by: jim

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1725504 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/STATUS b/STATUS
index b06328d399dfc74d68b7b1c8abc25e46cd33366e..557edd08e201bd18a0ac4ff5e318c241b81d046f 100644 (file)
--- a/STATUS
+++ b/STATUS
@@ -112,11 +112,6 @@ RELEASE SHOWSTOPPERS:
 PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
   [ start all new proposals below, under PATCHES PROPOSED. ]
 
-  *) core: Save memory when calling 'ap_parse_htaccess'
-     trunk patch: http://svn.apache.org/r1719016
-     2.4.x patch: trunk works
-     +1: jailletc36, covener, jim
-
   *) Use 'apr_pstrmemdup' instead of 'apr_pstrndup' when applicable in order
      to save a few cycles.
      Fix some style issues.

diff --git a/server/request.c b/server/request.c
index 849aad186326bfcf24594fba0a5b457aa64c4f96..f8a45d0b941de782e3b740ab0d4485b7c3fc6644 100644 (file)
--- a/server/request.c
+++ b/server/request.c
@@ -1012,8 +1012,7 @@ AP_DECLARE(int) ap_directory_walk(request_rec *r)
 
                 res = ap_parse_htaccess(&htaccess_conf, r, opts.override,
                                         opts.override_opts, opts.override_list,
-                                        apr_pstrdup(r->pool, r->filename),
-                                        sconf->access_name);
+                                        r->filename, sconf->access_name);
                 if (res) {
                     return res;
                 }