<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, May 26, 2021, 8:46 PM Gedare Bloom <<a href="mailto:gedare@rtems.org">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">On Wed, May 26, 2021 at 4:35 PM Harrison Edward Gerber<br>
<<a href="mailto:gerberhe11@gmail.com" target="_blank" rel="noreferrer">gerberhe11@gmail.com</a>> wrote:<br>
><br>
> See also CID 1399727<br>
><br>
> Closes #4444<br>
> ---<br>
>  cpukit/libmisc/monitor/mon-editor.c | 3 ++-<br>
>  1 file changed, 2 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/cpukit/libmisc/monitor/mon-editor.c b/cpukit/libmisc/monitor/mon-editor.c<br>
> index dcea9fcc69..1392be51f1 100644<br>
> --- a/cpukit/libmisc/monitor/mon-editor.c<br>
> +++ b/cpukit/libmisc/monitor/mon-editor.c<br>
> @@ -360,7 +360,8 @@ rtems_monitor_line_editor (<br>
>              {<br>
>                int bs;<br>
>                pos--;<br>
> -              strcpy (buffer + pos, buffer + pos + 1);<br>
> +              memmove(buffer + pos, buffer + pos + 1, RTEMS_COMMAND_BUFFER_SIZE - pos - 1);<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">My first thought was to ask if strlcpy would be more correct because it should avoid copying as many bytes. But the source and destination could overlap as they are close to it just by visual inspection. </div><div dir="auto"><br></div><div dir="auto">I think it would be worth a comment in the code explaining why a string method wasn't used.</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">
This exceeds 80 character line limit, please break the line length as<br>
directed by<br>
<a href="https://docs.rtems.org/branches/master/eng/coding-formatting.html#eighty-character-line-limit" rel="noreferrer noreferrer" target="_blank">https://docs.rtems.org/branches/master/eng/coding-formatting.html#eighty-character-line-limit</a><br>
<br>
> +              buffer[RTEMS_COMMAND_BUFFER_SIZE - 1] = "\0";<br>
This should be '\0' not "\0". I don't know if the compiler would<br>
correct that for you or not.<br>
<br>
>                fprintf(stdout,"\b%s \b", buffer + pos);<br>
>                for (bs = 0; bs < ((int) strlen (buffer) - pos); bs++)<br>
>                  putchar ('\b');<br>
> --<br>
> 2.25.1<br>
><br>
> _______________________________________________<br>
> devel mailing list<br>
> <a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
> <a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div></div>