Hi,
> From: Anatolij Gustschin <ag...@denx.de>
> Sent: dimanche 12 janvier 2020 20:51
>
> Hi Patrick,
>
> On Fri, 10 Jan 2020 08:50:31 +0100
> Patrick Delaunay patrick.delau...@st.com wrote:
> ...
> > diff --git a/tools/fdtgrep.c b/tools/fdtgrep.c index
> > 8f44f599c1..bcf06b871d 100644
> > --- a/tools/fdtgrep.c
> > +++ b/tools/fdtgrep.c
> > @@ -823,8 +823,10 @@ static int do_fdtgrep(struct display_info *disp, const
> char *filename)
> > }
> > if (count <= max_regions)
> > break;
> > + }
> > + if (count > max_regions) {
> > free(region);
>
> This change moved free(region) out of the loop. If you do so, then please also
> change malloc() in the loop to realloc():
>
> region = realloc(region, count * sizeof(struct fdt_region));
Yes, I will push V2 with realloc to avoid the memory leak that I introduced..
Thanks for review
> --
> Anatolij
Patrick
