[PATCH] dmi: dmicheck: zero buffer buf, cleans up a static analysis warning

Colin King colin.king at canonical.com
Fri Nov 10 17:48:31 UTC 2017


From: Colin Ian King <colin.king at canonical.com>

This is not strictly necessary, but it cleans up a static analysis warning
from CoverityScan.

Signed-off-by: Colin Ian King <colin.king at canonical.com>
---
 src/dmi/dmicheck/dmicheck.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/dmi/dmicheck/dmicheck.c b/src/dmi/dmicheck/dmicheck.c
index a55597f1..9bc8804c 100644
--- a/src/dmi/dmicheck/dmicheck.c
+++ b/src/dmi/dmicheck/dmicheck.c
@@ -306,6 +306,8 @@ static int dmi_load_file(const char* filename, void *buf, size_t size)
 	int fd;
 	ssize_t ret;
 
+	(void)memset(buf, 0, size);
+
 	if ((fd = open(filename, O_RDONLY)) < 0)
 		return FWTS_ERROR;
 	ret = read(fd, buf, size);
-- 
2.14.1




More information about the fwts-devel mailing list