backport of PR 59019 fix

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

diff --git a/CHANGES b/CHANGES
index fc31499e8a202406cababe48c12f15528808b4a2..91bad9827324476bd6147b98ba780b6c43af725d 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -2,6 +2,10 @@
 
 Changes with Apache 2.4.19
 
+  *) core: fix a bug in <UnDefine ...> directive processing. When used, the last
+     <Define...>'ed variable was also withdrawn. PR 59019
+     [Christophe Jaillet]
+
   *) mod_http2: Accept-Encoding is, when present on the initiating request, 
      added to push promises. This lets compressed content work in pushes.
      by the client. [Stefan Eissing]

diff --git a/STATUS b/STATUS
index 43dcff5817bd32e764d4e94dce28007983a70fbe..96672871d89d13bcad43e77ba2d630616f5a403e 100644 (file)
--- a/STATUS
+++ b/STATUS
@@ -112,14 +112,6 @@ RELEASE SHOWSTOPPERS:
 PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
   [ start all new proposals below, under PATCHES PROPOSED. ]
 
-  *) core: Fix a bug in <UnDefine ...> directive processing. When used, the
-           last <Define...>'ed variable was also withdrawn. PR 59019
-     Trunk patch:
-         http://svn.apache.org/r1732716
-     2.4.x patch:
-         Trunk version of patch works (modulo CHNAGES)
-     +1: jailletc36, ylavic, icing
-
   *) Save a few bytes in conf pool when parsing some directives
      Trunk patch:
          http://svn.apache.org/r1732252

diff --git a/server/core.c b/server/core.c
index 4dd8200d3668b1870836cffb0cedbd446ea74ea5..024bf8ddc38380171a8ec3307093f8fa9b86112d 100644 (file)
--- a/server/core.c
+++ b/server/core.c
@@ -1362,7 +1362,7 @@ static const char *unset_define(cmd_parms *cmd, void *dummy,
     defines = (char **)ap_server_config_defines->elts;
     for (i = 0; i < ap_server_config_defines->nelts; i++) {
         if (strcmp(defines[i], name) == 0) {
-            defines[i] = apr_array_pop(ap_server_config_defines);
+            defines[i] = *(char **)apr_array_pop(ap_server_config_defines);
             break;
         }
     }