On Tue, Aug 17, 2021 at 9:13 PM Ranier Vilela <ranier...@gmail.com> wrote: > > If that's conditions happen, all *result.index* touches are garbage. >
The patch looks valid to me, as the "index" member is not set in the "btp == NULL" case, and so has a junk value in the caller, and it's being used to index an array, BUT - isn't it also necessary to set the "split_pages" member to 0, because it also is not currently being set, and so too will have a junk value in this case (and it's possible for it to be referenced by the caller in this case). The "btp == NULL" case is not hit by any existing test cases, and does seem to be a rare case. Regards, Greg Nancarrow Fujitsu Australia