Re: cvs commit: httpd-2.0/server protocol.c

[Bill Stoddard]

I am fairly sure this patch is at least a contributing factor to a seg fault I am 
trying to debug.
The seg fault, which happens on the first request to the server,  is in ap_getline(). 
req_cfg is
NULL after the call to ap_get_module_config().  We blow up trying to reference 
req_cgg->bb. with a
NULL pointer. Here is the code snip...

    req_cfg = (core_request_config *)
                ap_get_module_config(r->request_config, &core_module);
    b = req_cfg->bb

The problem is that ap_set_module_config() has not been called on core_module yet.  
Ryan, did this
code serve pages for you before you committed it?

Bill



> rbb         01/04/17 20:53:35
>
>   Modified:    include  httpd.h
>                modules/http config.m4 http_core.c http_protocol.c
>                         mod_core.h
>                modules/loggers mod_log_config.c
>                server   protocol.c
>   Log:
>   Move the keepalives field out of the conn_rec and into an HTTP specific
>   connection record.  This also moves some HTTP specific back out of the
>   core and into the HTTP module.
>
>   Revision  Changes    Path
>   1.147     +0 -4      httpd-2.0/include/httpd.h
>