On 9/7/21 4:16 PM, Yann Ylavic wrote:
> 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 ;)
+1
Regards
Rüdiger
