From 0de4081d3b5689708136dd589d3554315e5ea6d1 Mon Sep 17 00:00:00 2001 From: Simon Howard Date: Fri, 27 Mar 2015 23:51:31 -0400 Subject: misc: Handle M_StringCopy() short buffer case. M_StringCopy() must always be provided a buffer of at least one character long, so that a NUL character can be written. If this is not the case, return immediately and indicate that the buffer was truncated. --- src/m_misc.c | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'src') diff --git a/src/m_misc.c b/src/m_misc.c index ab3c7009..53b86dba 100644 --- a/src/m_misc.c +++ b/src/m_misc.c @@ -379,6 +379,11 @@ boolean M_StringCopy(char *dest, const char *src, size_t dest_size) dest[dest_size - 1] = '\0'; strncpy(dest, src, dest_size - 1); } + else + { + return false; + } + len = strlen(dest); return src[len] == '\0'; } -- cgit v1.2.3