On 21 Oct 2021, at 12:57, Ruediger Pluem <[email protected]> wrote:

>>          *
>>          * Bump one byte off, and loop round to search again.
>>          */
>> -        apr_bucket_split(e, 1);
>> -        APR_BUCKET_REMOVE(e);
>> -        APR_BRIGADE_INSERT_TAIL(bbOut, e);
>> -
>> -        outbytes++;
>> +        ignore++;
> 
> Don't we need to reset ignore to 0 after each
> 
> APR_BUCKET_REMOVE(e);
> APR_BRIGADE_INSERT_TAIL(bbOut, e);
> 
> in the code above as we start with a new bucket then again and should use the 
> what we read from it from the beginning?

Talked myself out of needing this, but we do yes.

Regards,
Graham
—


Reply via email to