client controller/foreign_menu: use memmove instead of memcpy in readers

When src/dst memory areas may overlap, it's safer to use memmove.

1 files changed, 1 insertions, 1 deletions

diff --git a/client/foreign_menu.cpp b/client/foreign_menu.cpp
index 926e266f..00cc57ca 100644
--- a/client/foreign_menu.cpp
+++ b/client/foreign_menu.cpp
@@ -237,7 +237,7 @@ bool ForeignMenuConnection::read_msgs()
         pos += hdr->size;
     }
     if (nread > 0 && pos != _read_buf) {
-        memcpy(_read_buf, pos, nread);
+        memmove(_read_buf, pos, nread);
     }
     _read_pos = _read_buf + nread;
     return true;