On Sat, Aug 10, 2019 at 11:58:44AM +0200, Marion & Christophe JAILLET wrote: > Hi all, > > I would appreciate some other eyes on the patch below. > I guess that that the fix is correct, but I don't know the possible > implication of the fix. > > As said in the commit description, -1 seems to be a valid length, but I > don't know if such buckets can happen here.
(apr_size_t)-1 is the right way to compare here. I don't know of any case where indeterminate-length buckets will be present in input filtering, so I'd expect we never hit the -1 case in practice. But the change is right and it's a good find!
