<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 16 Mar 2021, 1:37 am Gedare Bloom, <<a href="mailto:gedare@rtems.org" target="_blank" rel="noreferrer">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This one looks ok to me, Niteesh?<br></blockquote></div></div><div dir="auto">+1</div><div dir="auto"><br></div><div dir="auto">The below question is not related to this patch.</div><div dir="auto"><br></div><div dir="auto">Out of interest, I started going through the code  and I am a bit confused in the following statement</div><div dir="auto"><br></div><div dir="auto"><span style="font-family:sans-serif"><a href="https://git.rtems.org/rtems/tree/cpukit/libmisc/rtems-fdt/rtems-fdt.c#n574">https://git.rtems.org/rtems/tree/cpukit/libmisc/rtems-fdt/rtems-fdt.c#n574</a></span><br></div><div dir="auto"><span style="font-family:sans-serif"><br></span></div><div dir="auto">This else block is reached when the FDT blob is not compressed, in this case the size variable is set to the file size since this is raw data.</div><div dir="auto"><br></div><div dir="auto">In the while loop, we are looping on size, but we never modify the size. I think, line 583 should be</div><div dir="auto">                    size -= r;</div><div dir="auto">Instead of</div><div dir="auto">                    r -= size;</div><div dir="auto">Is this a bug??</div><div dir="auto">Or am I missing something?</div><div dir="auto"><br></div><div dir="auto">PS: I am really sorry, If the formatting looks off I wrote this mail using my mobile.</div><div dir="auto"><br></div><div dir="auto">Thanks</div><div dir="auto">Niteesh</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On Fri, Mar 12, 2021 at 8:19 AM Ryan Long <<a href="mailto:ryan.long@oarcorp.com" rel="noreferrer noreferrer" target="_blank">ryan.long@oarcorp.com</a>> wrote:<br>
><br>
> CID 1437645: Resource leak in rtems_fdt_load().<br>
><br>
> Closes #4297<br>
> ---<br>
>  cpukit/libmisc/rtems-fdt/rtems-fdt.c | 1 +<br>
>  1 file changed, 1 insertion(+)<br>
><br>
> diff --git a/cpukit/libmisc/rtems-fdt/rtems-fdt.c b/cpukit/libmisc/rtems-fdt/rtems-fdt.c<br>
> index 0ea3653..5bb7ce0 100644<br>
> --- a/cpukit/libmisc/rtems-fdt/rtems-fdt.c<br>
> +++ b/cpukit/libmisc/rtems-fdt/rtems-fdt.c<br>
> @@ -611,6 +611,7 @@ rtems_fdt_load (const char* filename, rtems_fdt_handle* handle)<br>
>      return fe;<br>
>    }<br>
><br>
> +  close (bf);<br>
>    return 0;<br>
>  }<br>
><br>
> --<br>
> 1.8.3.1<br>
><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" rel="noreferrer noreferrer" target="_blank">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div></div>