Em ter., 17 de ago. de 2021 às 10:22, Greg Nancarrow <gregn4...@gmail.com>
escreveu:

> 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).
>
I agree.

Attached new version (v1).

regards,
Ranier Vilela

Attachment: fix_unitialized_var_index_freepage-v1.patch
Description: Binary data

Reply via email to