rpl...@apache.org wrote: > Author: rpluem > Date: Sat Jan 3 01:52:12 2009 > New Revision: 730949 > > URL: http://svn.apache.org/viewvc?rev=730949&view=rev > Log: > * Handle the case where conf->accf_map is NULL gracefully instead of > segfaulting > > Modified: > httpd/httpd/trunk/server/listen.c > > Modified: httpd/httpd/trunk/server/listen.c > URL: > http://svn.apache.org/viewvc/httpd/httpd/trunk/server/listen.c?rev=730949&r1=730948&r2=730949&view=diff > ============================================================================== > --- httpd/httpd/trunk/server/listen.c (original) > +++ httpd/httpd/trunk/server/listen.c Sat Jan 3 01:52:12 2009 > @@ -183,7 +183,7 @@ > const char* accf; > core_server_config *conf = ap_get_module_config(s->module_config, > &core_module); > - if (!proto) { > + if (!proto || !conf->accf_map) {
Ok, problem there. AIUI this should not happen. accf_map is created for the global-hosts (!is_virtual). All virtual hosts should have run the merge, which cpy's the global before fixing up specific fields in the merge. I'd prefer we revert and work backwards to where the bug really is.
