[PATCH v2] cpukit/libmisc/monitor: Fix src/dest overlap of strcpy in mon-editor.c
Harrison Edward Gerber
gerberhe11 at gmail.com
Fri May 28 00:31:53 UTC 2021
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
More information about the devel
mailing list