coregrind/m_gdbserver/remote-utils.c (prepare_resume_reply): Use memcpy
commitd7db69fbf482149e18ed1977df41fe320f5cbef2
authorMark Wielaard <mark@klomp.org>
Thu, 14 Nov 2024 11:25:27 +0000 (14 12:25 +0100)
committerMark Wielaard <mark@klomp.org>
Thu, 14 Nov 2024 11:42:38 +0000 (14 12:42 +0100)
treec26d79b62c04fcdf16c1e882864ec13dfba5ac7d
parent1698bd8d62fa6d3c9cf4628e289bb7aafb858136
coregrind/m_gdbserver/remote-utils.c (prepare_resume_reply): Use memcpy

GCC8 (but apparently not later versions) complain about the use of
strncpy when not actually copying a string:

remote-utils.c:1140:14: warning: 'char* strncpy(char*, const char*, size_t)' output truncated before terminating nul copying 6 bytes from a string of the same length [-Wstringop-truncation]
          strncpy (buf, "watch:", 6);
          ~~~~~~~~^~~~~~~~~~~~~~~~~~

This is "harmless" because buf is large enough and we will add more
chars (including a zero terminator) later. But using strncpy here is a
bit odd because we don't really want to copy a string, but an array of
6 chars. So use memcpy here to do so, simplyfing the code.
coregrind/m_gdbserver/remote-utils.c