Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2024-12-28 18:15:11 +00:00
Code Issues Projects Releases Packages Wiki Activity

[NTOSKRNL] Replace overlapping strcopy with memmove

Browse source 
Fixes 4 GCC 8 warnings of the kind:
ntoskrnl/kdbg/kdb_cli.c:3015:21: error: 'strcpy' accessing 1 byte at offsets 0 and [0, 2147483647] may overlap 1 byte at offset 0 [-Werror=restrict]
                     strcpy(p2, p2 + j);
                     ^~~~~~~~~~~~~~~~~~
This commit is contained in:
Timo Kreuzer 2019-04-28 20:23:35 +02:00
parent 95b3eebf71
commit 3af7cb825f
1 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
ntoskrnl/kdbg/kdb_cli.c
View file

@ -2662,15 +2662,16 @@ KdbpPrint(
{
while ((p2 = strrchr(p, '\x1b'))) /* Look for escape character */
{
size_t len = strlen(p2);
if (p2[1] == '[')
{
j = 2;
while (!isalpha(p2[j++]));
strcpy(p2, p2 + j);
memmove(p2, p2 + j, len + 1 - j);
}
else
{
strcpy(p2, p2 + 1);
memmove(p2, p2 + 1, len);
}
}
}
@ -3007,15 +3008,16 @@ KdbpPager(
{
while ((p2 = strrchr(p, '\x1b'))) /* Look for escape character */
{
size_t len = strlen(p2);
if (p2[1] == '[')
{
j = 2;
while (!isalpha(p2[j++]));
strcpy(p2, p2 + j);
memmove(p2, p2 + j, len + 1 - j);
}
else
{
strcpy(p2, p2 + 1);
memmove(p2, p2 + 1, len);
}
}
}