commit 247e6f088dbe19e5482e038009f45210586036c9
author Dave Rodgman <dave.rodgman@arm.com> Wed May 17 17:59:54 2023 +0100
committer GitHub <noreply@github.com> Wed May 17 17:59:54 2023 +0100
tree 213db62c51696c2bbc101086196f904608ef2c0d
parent 2710fe2d5e761f66d9695ca4bbb9b6a21458502a
parent bd771820637ee26a3df6bf9933fa59803fdaf595

Merge pull request #7608 from daverodgman/pr6511-2.28

Fix: no newline when debug msg over DEBUG_BUF_SIZE - backport 2.28

library/debug.c

diff --git a/library/debug.c b/library/debug.c
index ab8b352..3e794b5 100644
--- a/library/debug.c
+++ b/library/debug.c
@@ -30,6 +30,7 @@
 #include <stdio.h>
 #include <string.h>
 
+/* DEBUG_BUF_SIZE must be at least 2 */
 #define DEBUG_BUF_SIZE      512
 
 static int debug_threshold = 0;
@@ -69,6 +70,8 @@
     char str[DEBUG_BUF_SIZE];
     int ret = MBEDTLS_ERR_ERROR_CORRUPTION_DETECTED;
 
+    MBEDTLS_STATIC_ASSERT(DEBUG_BUF_SIZE >= 2, "DEBUG_BUF_SIZE too small");
+
     if (NULL == ssl              ||
         NULL == ssl->conf        ||
         NULL == ssl->conf->f_dbg ||
@@ -80,10 +83,15 @@
     ret = mbedtls_vsnprintf(str, DEBUG_BUF_SIZE, format, argp);
     va_end(argp);
 
-    if (ret >= 0 && ret < DEBUG_BUF_SIZE - 1) {
-        str[ret]     = '\n';
-        str[ret + 1] = '\0';
+    if (ret < 0) {
+        ret = 0;
+    } else {
+        if (ret >= DEBUG_BUF_SIZE - 1) {
+            ret = DEBUG_BUF_SIZE - 2;
+        }
     }
+    str[ret]     = '\n';
+    str[ret + 1] = '\0';
 
     debug_send_line(ssl, level, file, line, str);
 }