Commit 2baffd92 authored by Ryan Bloom's avatar Ryan Bloom
Browse files

Add back in logic to put the ending container directive in the tree. This 

needs to be there until we have a validating tree.  Without this, it is
possible for a config file to read:

<Ifmodule mod_foo.c>
    some directive
</Limit>

and still be read correctly.  When the tree building functions can validate
for us, this should be removed, and there is a comment that says so.


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

server/config.c

+7 −0
Changes for server/config.c: 7 added lines, 0 removed lines.
Original line number Diff line number Diff line
@@ -864,6 +864,13 @@ CORE_EXPORT(const char *) ap_handle_command(cmd_parms *parms, void *config, cons 
            current = ap_add_node(&curr_parent, current, newdir, 1);

        }

        else {

            /* The next line needs to be removed once we have a validating

             * tree building routine.

             * It is left in for now, so that we can ensure that

             * a container directive is followed by an appropriate closing

             * directive.

             */

            current = ap_add_node(&curr_parent, current, newdir, 0);

            current = curr_parent;

            curr_parent = current->parent;

        }