[PATCH v6 8/9] test: fix overflow warnings in mp_secondary and pdump

Stephen Hemminger stephen at networkplumber.org
Sun Nov 23 18:43:31 CET 2025


Both these tests lookup the current file prefix and
format up a filename argument. Avoid any possible format
overflows by allowing for bigger string.

Signed-off-by: Stephen Hemminger <stephen at networkplumber.org>
---
 app/test/test_mp_secondary.c | 4 ++--
 app/test/test_pdump.c        | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/app/test/test_mp_secondary.c b/app/test/test_mp_secondary.c
index f3694530a8..1b1d8b38f7 100644
--- a/app/test/test_mp_secondary.c
+++ b/app/test/test_mp_secondary.c
@@ -69,8 +69,8 @@ run_secondary_instances(void)
 	char coremask[10];
 
 #ifdef RTE_EXEC_ENV_LINUX
-	char tmp[PATH_MAX] = {0};
-	char prefix[PATH_MAX] = {0};
+	char tmp[PATH_MAX];
+	char prefix[PATH_MAX + sizeof("--file-prefix=")];
 
 	get_current_prefix(tmp, sizeof(tmp));
 
diff --git a/app/test/test_pdump.c b/app/test/test_pdump.c
index 9f7769707e..c809dd2aca 100644
--- a/app/test/test_pdump.c
+++ b/app/test/test_pdump.c
@@ -176,8 +176,8 @@ run_pdump_server_tests(void)
 	char coremask[10];
 
 #ifdef RTE_EXEC_ENV_LINUX
-	char tmp[PATH_MAX] = { 0 };
-	char prefix[PATH_MAX] = { 0 };
+	char tmp[PATH_MAX];
+	char prefix[PATH_MAX + sizeof("--file-prefix")];
 
 	get_current_prefix(tmp, sizeof(tmp));
 	snprintf(prefix, sizeof(prefix), "--file-prefix=%s", tmp);
-- 
2.51.0



More information about the dev mailing list