Force nul-termination of strings after vsnprintf().
authorGuus Sliepen <guus@tinc-vpn.org>
Thu, 23 Jun 2016 13:32:47 +0000 (15:32 +0200)
committerGuus Sliepen <guus@tinc-vpn.org>
Thu, 23 Jun 2016 13:32:47 +0000 (15:32 +0200)
Apparently, on Windows this function might not always be properly
terminated.

src/dropin.c
src/logger.c
src/protocol.c
src/xmalloc.c

index eb17acaad082ecb40d489bd5f7c0022cdadcd7d7..cd5563adb847414cf10d0440c58cad558b0f1ece 100644 (file)
@@ -140,6 +140,7 @@ int vasprintf(char **buf, const char *fmt, va_list ap) {
 
        va_copy(aq, ap);
        status = vsnprintf(*buf, len, fmt, aq);
+       buf[len - 1] = 0;
        va_end(aq);
 
        if(status >= 0)
index 6765cc5abeba7994b1283734714577eeb44f3bf1..e0e6bcad4714350bbcff38b3d45ae7631e386f52 100644 (file)
@@ -109,6 +109,7 @@ void logger(int priority, const char *format, ...) {
                                char message[4096];
                                const char *messages[] = {message};
                                vsnprintf(message, sizeof(message), format, ap);
+                               message[sizeof message - 1] = 0;
                                ReportEvent(loghandle, priority, 0, 0, NULL, 1, 0, messages, NULL);
                        }
 #else
index 5b8b7ba3372e9b40b0b4321f694ee49ce8564d38..fa169f32733550920c2760a5ca4ae027e6fd909c 100644 (file)
@@ -75,10 +75,11 @@ bool send_request(connection_t *c, const char *format, ...) {
           input buffer anyway */
 
        va_start(args, format);
-       len = vsnprintf(buffer, MAXBUFSIZE, format, args);
+       len = vsnprintf(buffer, sizeof buffer, format, args);
+       buffer[sizeof buffer - 1] = 0;
        va_end(args);
 
-       if(len < 0 || len > MAXBUFSIZE - 1) {
+       if(len < 0 || len > sizeof buffer - 1) {
                logger(LOG_ERR, "Output buffer overflow while sending request to %s (%s)",
                           c->name, c->hostname);
                return false;
index 39dc03c7229d79d7d321781ba78283fd1ea4f59e..a1b1fe8be87704d3cacdf37de7e2f728f1b9a2f8 100644 (file)
@@ -155,6 +155,7 @@ int xvasprintf(char **strp, const char *fmt, va_list ap) {
        int result = vsnprintf(buf, sizeof buf, fmt, ap);
        if(result < 0)
                exit(xalloc_exit_failure);
+       buf[sizeof buf - 1] = 0;
        *strp = xstrdup(buf);
 #else
        int result = vasprintf(strp, fmt, ap);