[PATCH v8 14/17] test: remove unneeded memset
Stephen Hemminger
stephen at networkplumber.org
Sun Feb 16 17:53:13 CET 2025
Since tmp is not used later in the function, this memset
is unnecessary. Even though this is harmless,
it causes tools that look for security issues
around memset to flag this a bug.
Signed-off-by: Stephen Hemminger <stephen at networkplumber.org>
Acked-by: Bruce Richardson <bruce.richardson at intel.com>
---
app/test/test_cmdline_cirbuf.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/app/test/test_cmdline_cirbuf.c b/app/test/test_cmdline_cirbuf.c
index f6361d49e9..37882b3779 100644
--- a/app/test/test_cmdline_cirbuf.c
+++ b/app/test/test_cmdline_cirbuf.c
@@ -279,8 +279,6 @@ test_cirbuf_string_add_del_reverse(void)
printf("Error: buffer should have been empty!\n");
return -1;
}
- /* clear tmp buffer */
- memset(tmp, 0, sizeof(tmp));
/*
* reinitialize circular buffer
--
2.47.2
More information about the dev
mailing list