test: fix format-truncation warnings
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v12 Latest8 patches 1 replyIn review
- v116 patches 0 repliesSuperseded
- v109 patches 8 repliesSuperseded
Earlier versions · 8
| Version | Posted | Patches | Replies | Status |
|---|---|---|---|---|
| v9 | 9 patches | 0 replies | Superseded | |
| v8 | 10 patches | 0 replies | Superseded | |
| v7 | 10 patches | 0 replies | Superseded | |
| v6 | 9 patches | 2 replies | Superseded | |
| v5 | 8 patches | 0 replies | Superseded | |
| v4 | 8 patches | 5 replies | Superseded | |
| v3 | 6 patches | 3 replies | Superseded | |
| v2 | 5 patches | 0 replies | Superseded |
Patches in v12 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/8 | test: increase size of memzone nameapp/ | +1 | 0 replies | |
| 2/8 | test: add new function to get current file-prefix argapp/ app/ | +71 | 0 replies | |
| 3/8 | test: avoid overflowing huge directory pathapp/ | +61 | 0 replies | |
| 4/8 | test: fix format overflow warning in ACL testapp/ | +14 | 0 replies | |
| 5/8 | test: fix impossible format-truncation in cfgfilesapp/ | +2 | 0 replies | |
| 6/8 | test: fix format overflow in cryptodev testapp/ | +18 | 0 replies | |
| 7/8 | test: fix overflow warnings in common codeapp/ app/ app/ | +12 | 0 replies | |
| 8/8 | test: re-enable format-truncation warningsapp/ | +0 | 0 replies |
Reviews trailers per patch
All 8 patches have a Reviewed-by or Acked-by.
| Patch | Bruce Richardson | Marat Khalili | Anatoly Burakov |
|---|---|---|---|
| 1/8 | A-b | ||
| 2/8 | A-b | ||
| 3/8 | A-b | ||
| 4/8 | A-b | ||
| 5/8 | A-b | A-b | |
| 6/8 | A-b | ||
| 7/8 | A-b | ||
| 8/8 | A-b | A-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.
Cover letter [PATCH v12 0/8] test: fix format-truncation warnings
This patchset addresses the -Wformat-truncation warnings throughout the DPDK test suite, allowing the warning to be re-enabled.
The changes include:
- Properly sizing buffers using named constants (RTE_MEMZONE_NAMESIZE, NAME_MAX)
- Adding a common file_prefix_arg() helper to reduce code duplication
- Using standard library functions (getmntent) for mount parsing
- Checking snprintf() return values for potential truncation
- Converting problematic 2D arrays to pointer arrays
After these fixes, the final patch removes -Wno-format-truncation from the test build flags, ensuring these issues are caught in the future.
Several patches are marked for stable backport where they fix real potential bugs rather than just theoretical compiler warnings.
v12 - rebase to handle new eal_flags printf's
Stephen Hemminger (8):
test: increase size of memzone name
test: add new function to get current file-prefix arg
test: avoid overflowing huge directory path
test: fix format overflow warning in ACL test
test: fix impossible format-truncation in cfgfiles
test: fix format overflow in cryptodev test
test: fix overflow warnings in common code
test: re-enable format-truncation warnings
app/test/meson.build | 1 - app/test/process.h | 44 ++++-- app/test/test_cfgfile.c | 8 +- app/test/test_cryptodev.c | 22 ++- app/test/test_eal_flags.c | 269 +++++++++++--------------------- app/test/test_memzone.c | 2 +- app/test/test_mp_secondary.c | 14 +- app/test/test_pdump.c | 13 +- app/test/test_table_acl.c | 30 ++-- app/test/test_timer_secondary.c | 12 +- 10 files changed, 174 insertions(+), 241 deletions(-)
Signature · 1 line
2.51.0