Bananymous
/
banan-os
1
1
Fork 1
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

BAN: Use memmove instead of memcpy on overlapping data 

I was accidentally using memcpy where memmove was needed
This commit is contained in:
Bananymous 2024-10-13 21:58:44 +03:00
parent 1c1a76d6d7
commit 0b05e9827b
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
BAN/include/BAN/String.h
View File

@ -127,7 +127,7 @@ namespace BAN
void remove(size_type index) void remove(size_type index)
{ {
ASSERT(index < m_size); ASSERT(index < m_size);
memcpy(data() + index, data() + index + 1, m_size - index); memmove(data() + index, data() + index + 1, m_size - index);
m_size--; m_size--;
data()[m_size] = '\0'; data()[m_size] = '\0';
} }