From: Stas Bekman <stas@apache.org>
Date: Sat, 11 Oct 2003 06:37:45 +0000 (+0000)
Subject: fix the config parser to support <Foo>..</Foo> containers (no
X-Git-Tag: pre_ajp_proxy~1115
X-Git-Url: http://git.ipfire.org/gitweb/index.cgi?a=commitdiff_plain;h=19eb03c5a6abac6f2aeeba4ff4325478186f43d1;p=thirdparty%2Fapache%2Fhttpd.git

fix the config parser to support <Foo>..</Foo> containers (no
arguments in the opening tag) supported by httpd 1.3. Without
this change mod_perl 2.0's <Perl> sections are broken.
PR:
Obtained from:
Submitted by:	"Philippe M. Chiasson" <gozer@cpan.org>
Reviewed by:	stas


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

diff --git a/CHANGES b/CHANGES
index ffd805665b1..737aa89d320 100644
--- a/CHANGES
+++ b/CHANGES
@@ -2,6 +2,11 @@ Changes with Apache 2.1.0-dev
 
   [Remove entries to the current 2.0 section below, when backported]
 
+  *) fix the config parser to support <Foo>..</Foo> containers (no
+     arguments in the opening tag) supported by httpd 1.3. Without
+     this change mod_perl 2.0's <Perl> sections are broken.
+     ["Philippe M. Chiasson" <gozer@cpan.org>]
+
   *) mod_cgid: fix a hash table corruption problem which could
      result in the wrong script being cleaned up at the end of a
      request.  [Jeff Trawick]
diff --git a/STATUS b/STATUS
index 2856232b482..56f1d2f809b 100644
--- a/STATUS
+++ b/STATUS
@@ -1,5 +1,5 @@
 APACHE 2.1 STATUS:                                              -*-text-*-
-Last modified at [$Date: 2003/08/31 16:14:38 $]
+Last modified at [$Date: 2003/10/11 06:37:45 $]
 
 Release [NOTE that only Alpha/Beta releases occur in 2.1 development]:
 
@@ -30,6 +30,13 @@ RELEASE SHOWSTOPPERS:
 
 CURRENT VOTES:
 
+    * httpd-2.0's config parser is incompatible with httpd-1.3's one,
+      which allowed containers like <Perl>. httpd-2.0's config parser
+      doesn't like container directives with no arguments (Syntax
+      error on line nn of httpd.conf: <Perl> directive missing closing
+      '>'). This fix restores the support for <Foo>..</Foo> containers.
+      +1: stas
+
     * Promote mod_cache from experimental to non-experimental
       status (keep issues noted below in EXPERIMENTAL MODULES as
       items to be addressed as a supported module).
diff --git a/server/config.c b/server/config.c
index 9a8a7285334..7a3ccb03e2b 100644
--- a/server/config.c
+++ b/server/config.c
@@ -926,6 +926,9 @@ static const char *ap_build_config_sub(apr_pool_t *p, apr_pool_t *temp_pool,
         if (*lastc == '>') {
             *lastc = '\0' ;
         }
+        if (cmd_name[0] == '<' && *args == '\0') {
+            args = ">";
+        }
     }
 
     newdir = apr_pcalloc(p, sizeof(ap_directive_t));