On Tue, Sep 7, 2021 at 3:05 PM Ruediger Pluem <rpl...@apache.org> wrote: > > On 9/7/21 2:00 PM, Yann Ylavic wrote: > > On Mon, May 25, 2020 at 7:50 AM <rpl...@apache.org> wrote: > >> > >> Modified: httpd/httpd/trunk/server/scoreboard.c > >> URL: > >> http://svn.apache.org/viewvc/httpd/httpd/trunk/server/scoreboard.c?rev=1878092&r1=1878091&r2=1878092&view=diff > >> ============================================================================== > >> --- httpd/httpd/trunk/server/scoreboard.c (original) > >> +++ httpd/httpd/trunk/server/scoreboard.c Mon May 25 05:50:12 2020 > >> @@ -381,7 +381,7 @@ AP_DECLARE(void) ap_increment_counts(ap_ > >> if (pfn_ap_logio_get_last_bytes != NULL) { > >> bytes = pfn_ap_logio_get_last_bytes(r->connection); > >> } > >> - else if (r->method_number == M_GET && r->method[0] == 'H') { > >> + else if (r->method_number == M_GET && r->method && r->method[0] == > >> 'H') { > >> bytes = 0; > >> } > >> else { > > > > Sorry for the lateness.. > > Maybe we could have an r->method_number == M_INVALID and r->method == > > "" by default on failure, like with the attached patch? > > Instead of the above or on top?
I'd say on top, the above can't hurt ;) Regards; Yann.