All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stephen Hemminger <stephen@networkplumber.org>
To: dev@dpdk.org
Cc: Stephen Hemminger <stephen@networkplumber.org>,
	Bruce Richardson <bruce.richardson@intel.com>,
	Anatoly Burakov <anatoly.burakov@intel.com>
Subject: [PATCH v9 1/9] test: increase size of memzone name
Date: Mon,  1 Dec 2025 14:01:59 -0800	[thread overview]
Message-ID: <20251201220421.207285-2-stephen@networkplumber.org> (raw)
In-Reply-To: <20251201220421.207285-1-stephen@networkplumber.org>

The tests were using a name buffer of 20 characters which could
overflow if number of memory zones got large. The upper limit
is defined as RTE_MEMZONE_NAMESIZE so use that.

In function ‘test_memzone_free’,
    inlined from ‘test_memzone’ at ../app/test/test_memzone.c:1117:6:
../app/test/test_memzone.c:52:35: warning: ‘%u’ directive output may be truncated writing between 1 and 10 bytes into a region of size 4 [-Wformat-truncation=]
   52 | #define TEST_MEMZONE_NAME(suffix) "MZ_TEST_" suffix
      |                                   ^~~~~~~~~~
../app/test/test_memzone.c:931:46: note: in expansion of macro ‘TEST_MEMZONE_NAME’
  931 |                 snprintf(name, sizeof(name), TEST_MEMZONE_NAME("tempzone%u"),
      |                                              ^~~~~~~~~~~~~~~~~
../app/test/test_memzone.c:52:35: note: directive argument in the range [0, 2147483647]
   52 | #define TEST_MEMZONE_NAME(suffix) "MZ_TEST_" suffix
      |                                   ^~~~~~~~~~
../app/test/test_memzone.c:931:46: note: in expansion of macro ‘TEST_MEMZONE_NAME’
  931 |                 snprintf(name, sizeof(name), TEST_MEMZONE_NAME("tempzone%u"),
      |                                              ^~~~~~~~~~~~~~~~~
../app/test/test_memzone.c:931:17: note: ‘snprintf’ output between 18 and 27 bytes into a destination of size 20
  931 |                 snprintf(name, sizeof(name), TEST_MEMZONE_NAME("tempzone%u"),
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  932 |                                 i);
      |

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
---
 app/test/test_memzone.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/test/test_memzone.c b/app/test/test_memzone.c
index 506725ea41..6586dad18d 100644
--- a/app/test/test_memzone.c
+++ b/app/test/test_memzone.c
@@ -872,7 +872,7 @@ test_memzone_free(void)
 {
 	const struct rte_memzone **mz;
 	int i;
-	char name[20];
+	char name[RTE_MEMZONE_NAMESIZE];
 	int rc = -1;
 
 	mz = rte_calloc("memzone_test", rte_memzone_max_get() + 1,
-- 
2.51.0


  reply	other threads:[~2025-12-01 22:04 UTC|newest]

Thread overview: 120+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-11-10 18:19 [PATCH 0/3] examples: format truncation bugs Stephen Hemminger
2025-11-10 18:19 ` [PATCH 1/3] examples/server_node_efd: fix format overflow Stephen Hemminger
2025-11-10 18:19 ` [PATCH 2/3] examples/vdpa: fix format overflow warning Stephen Hemminger
2025-11-10 18:19 ` [PATCH 3/3] examples: re-enable format truncation warning Stephen Hemminger
2025-11-11 22:17 ` [PATCH v2 0/7] fix format-truncation warnings Stephen Hemminger
2025-11-11 22:17   ` [PATCH v2 1/7] examples/server_node_efd: fix format overflow Stephen Hemminger
2025-11-11 22:17   ` [PATCH v2 2/7] examples/vdpa: fix format overflow warning Stephen Hemminger
2025-11-14 15:59     ` Bruce Richardson
2025-11-11 22:17   ` [PATCH v2 3/7] examples/ip_reassembly: add check before formatting name Stephen Hemminger
2025-11-12 14:50     ` Konstantin Ananyev
2025-11-14 16:05     ` Bruce Richardson
2025-11-11 22:17   ` [PATCH v2 4/7] examples: re-enable format truncation warning Stephen Hemminger
2025-11-14 16:07     ` Bruce Richardson
2025-11-11 22:17   ` [PATCH v2 5/7] test: refactor file prefix arg handling Stephen Hemminger
2025-11-11 22:17   ` [PATCH v2 6/7] test: increase size of memzone name Stephen Hemminger
2025-11-14 16:07     ` Bruce Richardson
2025-11-11 22:17   ` [PATCH v2 7/7] test: re-enable format-truncation warnings Stephen Hemminger
2025-11-14 16:08     ` Bruce Richardson
2025-11-15 19:36 ` [PATCH v3 0/4] examples: fix " Stephen Hemminger
2025-11-15 19:36   ` [PATCH v3 1/4] examples/server_node_efd: fix format overflow Stephen Hemminger
2025-11-17  8:50     ` Bruce Richardson
2025-11-15 19:36   ` [PATCH v3 2/4] examples/vdpa: fix format overflow warning Stephen Hemminger
2025-11-19  4:31     ` Thomas Monjalon
2025-11-19  9:13       ` Bruce Richardson
2025-11-15 19:36   ` [PATCH v3 3/4] examples/ip_reassembly: add check before formatting name Stephen Hemminger
2025-11-15 19:36   ` [PATCH v3 4/4] examples: enable format truncation warning Stephen Hemminger
2025-11-20 16:21 ` [PATCH v4 0/4] examples: fix format-truncation errors Stephen Hemminger
2025-11-20 16:21   ` [PATCH v4 1/4] examples/server_node_efd: fix format overflow Stephen Hemminger
2025-11-20 16:22   ` [PATCH v4 2/4] examples/vdpa: fix format overflow warning Stephen Hemminger
2025-11-20 16:22   ` [PATCH v4 3/4] examples/ip_reassembly: add check before formatting name Stephen Hemminger
2025-11-20 16:22   ` [PATCH v4 4/4] examples: enable format truncation warning Stephen Hemminger
2025-11-24 16:20   ` [PATCH v4 0/4] examples: fix format-truncation errors David Marchand
2025-11-21 17:08 ` [PATCH v5 0/8] tests: fix format truncation warnings Stephen Hemminger
2025-11-21 17:08   ` [PATCH v5 1/8] test: increase size of memzone name Stephen Hemminger
2025-11-21 17:08   ` [PATCH v5 2/8] test: refactor file prefix arg handling Stephen Hemminger
2025-11-21 17:08   ` [PATCH v5 3/8] test: avoid overflowing huge directory path Stephen Hemminger
2025-11-21 17:08   ` [PATCH v5 4/8] test: use unit test runner for eal flags Stephen Hemminger
2025-11-21 17:08   ` [PATCH v5 5/8] test: fix format overflow warning in ACL test Stephen Hemminger
2025-11-21 17:08   ` [PATCH v5 6/8] test: fix impossible format-truncation in cfgfiles Stephen Hemminger
2025-11-21 17:08   ` [PATCH v5 7/8] test: fix format overflow in cryptodev test Stephen Hemminger
2025-11-21 17:08   ` [PATCH v5 8/8] test: re-enable format-truncation warnings Stephen Hemminger
2025-11-23 17:43 ` [PATCH v6 0/9] tests: fix format truncation warnings Stephen Hemminger
2025-11-23 17:43   ` [PATCH v6 1/9] test: increase size of memzone name Stephen Hemminger
2025-11-23 17:43   ` [PATCH v6 2/9] test: refactor file prefix arg handling Stephen Hemminger
2025-11-23 17:43   ` [PATCH v6 3/9] test: avoid overflowing huge directory path Stephen Hemminger
2025-11-23 17:43   ` [PATCH v6 4/9] test: use unit test runner for eal flags Stephen Hemminger
2025-11-23 17:43   ` [PATCH v6 5/9] test: fix format overflow warning in ACL test Stephen Hemminger
2025-11-23 17:43   ` [PATCH v6 6/9] test: fix impossible format-truncation in cfgfiles Stephen Hemminger
2025-11-23 17:43   ` [PATCH v6 7/9] test: fix format overflow in cryptodev test Stephen Hemminger
2025-11-23 17:43   ` [PATCH v6 8/9] test: fix overflow warnings in mp_secondary and pdump Stephen Hemminger
2025-11-23 17:43   ` [PATCH v6 9/9] test: re-enable format-truncation warnings Stephen Hemminger
2025-11-24  1:53   ` [PATCH v6 0/9] tests: fix format truncation warnings fengchengwen
2025-11-24 15:45   ` Patrick Robb
2025-11-25 15:20 ` [PATCH v7 00/10] test: " Stephen Hemminger
2025-11-25 15:20   ` [PATCH v7 01/10] test: increase size of memzone name Stephen Hemminger
2025-11-25 15:20   ` [PATCH v7 02/10] test: add new function to get current file-prefix arg Stephen Hemminger
2025-11-25 15:20   ` [PATCH v7 03/10] test: refactor file prefix arg handling Stephen Hemminger
2025-11-25 15:20   ` [PATCH v7 04/10] test: avoid overflowing huge directory path Stephen Hemminger
2025-11-25 15:20   ` [PATCH v7 05/10] test: fix format overflow warning in ACL test Stephen Hemminger
2025-11-25 15:20   ` [PATCH v7 06/10] test: fix impossible format-truncation in cfgfiles Stephen Hemminger
2025-11-25 15:20   ` [PATCH v7 07/10] test: fix format overflow in cryptodev test Stephen Hemminger
2025-11-25 15:20   ` [PATCH v7 08/10] test: fix overflow warnings in several places by using common code Stephen Hemminger
2025-11-25 15:20   ` [PATCH v7 09/10] test: use unit test runner for eal flags Stephen Hemminger
2025-11-25 15:21   ` [PATCH v7 10/10] test: re-enable format-truncation warnings Stephen Hemminger
2025-11-28 22:42 ` [PATCH v8 00/10] test: cleanup and format truncation fixes Stephen Hemminger
2025-11-28 22:42   ` [PATCH v8 01/10] test: increase size of memzone name Stephen Hemminger
2025-11-28 22:42   ` [PATCH v8 02/10] test: add new function to get current file-prefix arg Stephen Hemminger
2025-11-28 22:42   ` [PATCH v8 03/10] test: refactor file prefix arg handling Stephen Hemminger
2025-11-28 22:42   ` [PATCH v8 04/10] test: avoid overflowing huge directory path Stephen Hemminger
2025-11-28 22:42   ` [PATCH v8 05/10] test: fix format overflow warning in ACL test Stephen Hemminger
2025-11-28 22:42   ` [PATCH v8 06/10] test: fix impossible format-truncation in cfgfiles Stephen Hemminger
2025-11-28 22:42   ` [PATCH v8 07/10] test: fix format overflow in cryptodev test Stephen Hemminger
2025-11-28 22:42   ` [PATCH v8 08/10] test: fix overflow warnings in several places by using common code Stephen Hemminger
2025-11-28 22:42   ` [PATCH v8 09/10] test: use unit test runner for eal flags Stephen Hemminger
2025-11-28 22:42   ` [PATCH v8 10/10] test: re-enable format-truncation warnings Stephen Hemminger
2025-12-01 22:01 ` [PATCH v9 0/9] test: cleanup and format truncation fixes Stephen Hemminger
2025-12-01 22:01   ` Stephen Hemminger [this message]
2025-12-01 22:02   ` [PATCH v9 2/9] test: add new function to get current file-prefix arg Stephen Hemminger
2025-12-01 22:02   ` [PATCH v9 3/9] test: refactor file prefix arg handling Stephen Hemminger
2025-12-01 22:02   ` [PATCH v9 4/9] test: avoid overflowing huge directory path Stephen Hemminger
2025-12-01 22:02   ` [PATCH v9 5/9] test: fix format overflow warning in ACL test Stephen Hemminger
2025-12-01 22:02   ` [PATCH v9 6/9] test: fix impossible format-truncation in cfgfiles Stephen Hemminger
2025-12-01 22:02   ` [PATCH v9 7/9] test: fix format overflow in cryptodev test Stephen Hemminger
2025-12-01 22:02   ` [PATCH v9 8/9] test: fix overflow warnings in several places by using common code Stephen Hemminger
2025-12-01 22:02   ` [PATCH v9 9/9] test: re-enable format-truncation warnings Stephen Hemminger
2025-12-29 21:59 ` [PATCH v10 0/9] tests: enable and fix format overflow Stephen Hemminger
2025-12-29 21:59   ` [PATCH v10 1/9] test: increase size of memzone name Stephen Hemminger
2025-12-29 21:59   ` [PATCH v10 2/9] test: add new function to get current file-prefix arg Stephen Hemminger
2026-01-02 12:50     ` Marat Khalili
2026-01-02 17:10       ` Stephen Hemminger
2025-12-29 21:59   ` [PATCH v10 3/9] test: refactor file prefix arg handling Stephen Hemminger
2026-01-02 14:03     ` Marat Khalili
2025-12-29 21:59   ` [PATCH v10 4/9] test: avoid overflowing huge directory path Stephen Hemminger
2026-01-02 14:27     ` Marat Khalili
2025-12-29 21:59   ` [PATCH v10 5/9] test: fix format overflow warning in ACL test Stephen Hemminger
2026-01-02 14:33     ` Marat Khalili
2025-12-29 21:59   ` [PATCH v10 6/9] test: fix impossible format-truncation in cfgfiles Stephen Hemminger
2026-01-02 14:47     ` Marat Khalili
2025-12-29 21:59   ` [PATCH v10 7/9] test: fix format overflow in cryptodev test Stephen Hemminger
2025-12-29 21:59   ` [PATCH v10 8/9] test: fix overflow warnings in common code Stephen Hemminger
2026-01-02 14:54     ` Marat Khalili
2025-12-29 21:59   ` [PATCH v10 9/9] test: re-enable format-truncation warnings Stephen Hemminger
2026-01-02 14:55     ` Marat Khalili
2026-01-17  2:36 ` [PATCH v11 0/6] tests: enable format-overflow and fix warnings Stephen Hemminger
2026-01-17  2:36   ` [PATCH v11 1/6] test: increase size of memzone name Stephen Hemminger
2026-01-17  2:36   ` [PATCH v11 2/6] test: add new function to get current file-prefix arg Stephen Hemminger
2026-01-17  2:36   ` [PATCH v11 3/6] test: avoid overflowing huge directory path Stephen Hemminger
2026-01-17  2:36   ` [PATCH v11 4/6] test: fix format overflow warning in ACL test Stephen Hemminger
2026-01-17  2:37   ` [PATCH v11 5/6] test: fix impossible format-truncation in cfgfiles Stephen Hemminger
2026-01-17  2:37   ` [PATCH v11 6/6] test: fix format overflow in cryptodev test Stephen Hemminger
2026-01-21 21:06 ` [PATCH v12 0/8] test: fix format-truncation warnings Stephen Hemminger
2026-01-21 21:06   ` [PATCH v12 1/8] test: increase size of memzone name Stephen Hemminger
2026-01-21 21:06   ` [PATCH v12 2/8] test: add new function to get current file-prefix arg Stephen Hemminger
2026-01-21 21:06   ` [PATCH v12 3/8] test: avoid overflowing huge directory path Stephen Hemminger
2026-01-21 21:06   ` [PATCH v12 4/8] test: fix format overflow warning in ACL test Stephen Hemminger
2026-01-21 21:06   ` [PATCH v12 5/8] test: fix impossible format-truncation in cfgfiles Stephen Hemminger
2026-01-21 21:06   ` [PATCH v12 6/8] test: fix format overflow in cryptodev test Stephen Hemminger
2026-01-21 21:06   ` [PATCH v12 7/8] test: fix overflow warnings in common code Stephen Hemminger
2026-01-21 21:06   ` [PATCH v12 8/8] test: re-enable format-truncation warnings Stephen Hemminger
2026-02-06 11:21   ` [PATCH v12 0/8] test: fix " David Marchand

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20251201220421.207285-2-stephen@networkplumber.org \
    --to=stephen@networkplumber.org \
    --cc=anatoly.burakov@intel.com \
    --cc=bruce.richardson@intel.com \
    --cc=dev@dpdk.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.