AFAICT, we have never created server configs of _all_ modules for _all_

  contexts, only when they come up (e.g., a directive of that particular
  module is provided) with the except that vhost containers complete the
  initialization of all modules.  I haven't stepped this, but it appears
  to be correct.

  This patch assures we can merge two incomplete directory sections
  together.  They will (obviously) need to be merged with a LHS default,
  e.g. a vhost's or main server's default dir config.  That's fine, this
  just allows either the LHS or RHS value to be null, and for merging to
  still succeed.

Reported by:  Sander Striker <striker@apache.org>
Additional Analysis:  Doug MacEachern

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

diff --git a/server/config.c b/server/config.c
index dc390eb89659e0e237a0a484fa6335e2b913263b..4258f31bcd07af6d59dfd9deb79d468671b9c837 100644 (file)
--- a/server/config.c
+++ b/server/config.c
@@ -258,7 +258,7 @@ AP_CORE_DECLARE(ap_conf_vector_t*) ap_merge_per_dir_configs(apr_pool_t *p,
        merger_func df = modp->merge_dir_config;
        int i = modp->module_index;
 
-       if (df && new_vector[i])
+       if (df && new_vector[i] && base_vector[i])
            conf_vector[i] = (*df) (p, base_vector[i], new_vector[i]);
        else
            conf_vector[i] = new_vector[i] ? new_vector[i] : base_vector[i];