Return-Path: Delivered-To: apmail-httpd-dev-archive@httpd.apache.org Received: (qmail 19232 invoked by uid 500); 10 Sep 2001 12:42:53 -0000 Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm Precedence: bulk Reply-To: dev@httpd.apache.org list-help: list-unsubscribe: list-post: Delivered-To: mailing list dev@httpd.apache.org Received: (qmail 19050 invoked from network); 10 Sep 2001 12:42:51 -0000 Errors-To: Message-ID: <04d701c139f5$a9dfc5f0$93c0b0d0@roweclan.net> From: "William A. Rowe, Jr." To: Cc: References: Subject: Re: mod_ssl broken Date: Mon, 10 Sep 2001 07:36:37 -0500 MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit X-Priority: 3 X-MSMail-Priority: Normal X-Mailer: Microsoft Outlook Express 5.50.4133.2400 X-MimeOLE: Produced By Microsoft MimeOLE V5.50.4133.2400 X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N From: "Sander Striker" Sent: Monday, September 10, 2001 3:24 AM > >> I've looked into this slightly, and the underlying problem is that > >> ap_merge_per_dir_configs finds a previous cached merge (for /?) and > >> tries to use it but the entry for mod_ssl is NULL. The URL, btw, is > >> /svn. > > > > This is very, very odd. Would someone please post the svn > > create/merge dir config > > code, just in case it's clobbering a sibling? > > #define INHERIT_VALUE(parent, child, field) \ > ((child)->field ? (child)->field : (parent)->field) > > Looks ok to me... Looks OK to me too... Who wants to toss together a patch to pull all the helper code from mod_ssl, change it from eastern to western European, and drop it back in the right place (AP_ decorated) such as ap_config.h? If _we_ need these sort of helpers... #define cfgMerge(el,unset) new->el = (add->el == (unset)) ? base->el : add->el #define cfgMergeArray(el) new->el = apr_array_append(p, add->el, base->el) #define cfgMergeTable(el) new->el = apr_table_overlay(p, add->el, base->el) #define cfgMergeCtx(el) new->el = apr_table_overlay(p, add->el, base->el) #define cfgMergeString(el) cfgMerge(el, NULL) #define cfgMergeBool(el) cfgMerge(el, UNSET) #define cfgMergeInt(el) cfgMerge(el, UNSET) then apparently (as your example shows) the rest of the world needs these as well. Bill