[PATCH v2] cpukit/libmisc/monitor: Fix src/dest overlap of strcpy in mon-editor.c

Vijay Kumar Banerjee vijay at rtems.org
Fri May 28 22:01:06 UTC 2021


On Fri, May 28, 2021 at 1:08 PM Joel Sherrill <joel at rtems.org> wrote:
>
> This looks OK.

Pushed. Thanks.
>
> On Thu, May 27, 2021 at 7:32 PM Harrison Edward Gerber <gerberhe11 at gmail.com> wrote:
>>
>> From: Harrison Edward Gerber <gerberhe11 at gmail.com>
>>
>> See also CID 1399727
>>
>> Closes #4444
>> ---
>>  cpukit/libmisc/monitor/mon-editor.c | 12 +++++++++++-
>>  1 file changed, 11 insertions(+), 1 deletion(-)
>>
>> diff --git a/cpukit/libmisc/monitor/mon-editor.c b/cpukit/libmisc/monitor/mon-editor.c
>> index dcea9fcc69..4287b399a5 100644
>> --- a/cpukit/libmisc/monitor/mon-editor.c
>> +++ b/cpukit/libmisc/monitor/mon-editor.c
>> @@ -360,7 +360,17 @@ rtems_monitor_line_editor (
>>              {
>>                int bs;
>>                pos--;
>> -              strcpy (buffer + pos, buffer + pos + 1);
>> +
>> +              /*
>> +               * Memory operation used here instead of string
>> +               * method due the src and dest of buffer overlapping.
>> +               */
>> +              memmove(
>> +                buffer + pos,
>> +                buffer + pos + 1,
>> +                RTEMS_COMMAND_BUFFER_SIZE - pos - 1
>> +              );
>> +              buffer[RTEMS_COMMAND_BUFFER_SIZE - 1] = '\0';
>>                fprintf(stdout,"\b%s \b", buffer + pos);
>>                for (bs = 0; bs < ((int) strlen (buffer) - pos); bs++)
>>                  putchar ('\b');
>> --
>> 2.25.1
>>
>> _______________________________________________
>> devel mailing list
>> devel at rtems.org
>> http://lists.rtems.org/mailman/listinfo/devel
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel


More information about the devel mailing list