Silence strncpy warning emitted by GCC 8 in ini parser

While copying 'dst len' bytes in strncpy is normally risky
as the dst may not be NULL-terminated, this function ensures
that the last byte of 'dst' is NULL.

Therefore, this change is mostly made to silence GCC.

Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>

diff --git a/src/common/config/ini.c b/src/common/config/ini.c
index 34e3abf8a682a9bb8b3fe048b2bb8ecd6741c534..41d299a19be5343bb12518db407422468c05baf0 100644 (file)
--- a/src/common/config/ini.c
+++ b/src/common/config/ini.c
@@ -82,7 +82,7 @@ static char* find_char_or_comment(const char* s, char c)
 /* Version of strncpy that ensures dest (size bytes) is null-terminated. */
 static char* strncpy0(char* dest, const char* src, size_t size)
 {
-       strncpy(dest, src, size);
+       strncpy(dest, src, size - 1);
        dest[size - 1] = '\0';
        return dest;
 }