<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Oct 15, 2020 at 2:13 PM Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 15/10/20 10:27 pm, Frank Kuehndel wrote:<br>
> From: Frank Kühndel <<a href="mailto:frank.kuehndel@embedded-brains.de" target="_blank">frank.kuehndel@embedded-brains.de</a>><br>
> <br>
> The compiler warning was:<br>
> <br>
> ../../../cpukit/libmisc/rtems-fdt/rtems-fdt.c:267:5: warning:<br>
> 'strncpy' specified bound depends on the length of the source argument<br>
>   267 |     strncpy(path, name, namelen);<br>
>       |     ^~~~~~~~~~~~~~~~~~~~~~~~~~~~<br>
> <br>
> It turns out that the `strncpy()` nor the buffer `path` is needed when<br>
> one uses `strncmp()` instead of `strcmp()`. This needs some change to<br>
> the algorithm but has the advantage that `name` is never truncated<br>
> to the size of the buffer `path`.<br>
> <br>
> Note:<br>
> <br>
> rtems-fdt.c, rtems-fdt-shell.c and cpukit/include/rtems/rtems-fdt.h<br>
> seem to be dead code.<br>
<br>
We cannot tell this is as a user may depend on it. With this command I know<br>
users on Zynq platforms that use FDT to map PL logic to drivers. The command is<br>
very much in use and is used as a PS to PL testing method.<br>
<br>
> They implement a shell command `fdt` but that<br>
> command is not part of the shell nor of any macro in<br>
> cpukit/include/rtems/shellconfig.h.<br>
<br>
It have been missed and needs be added. Adding something in shellconfig.h would<br>
be a help.<br></blockquote><div><br></div><div>Does that mean it also is not in the Shell manual?</div><div><br></div><div>--joel </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Thanks<br>
Chris<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a></blockquote></div></div>