From ea2d0ecf08842d52f77d1a88a184a85960999e1e Mon Sep 17 00:00:00 2001 From: Victor Julien Date: Tue, 26 Apr 2022 20:01:19 +0200 Subject: [PATCH] datasets: fix cppcheck warning src/datasets.c:107:17: error: Uninitialized variable: hash [uninitvar] memcpy(out, hash, outs); ^ src/datasets.c:93:26: note: Assuming condition is false for (x = 0, i = 0; i < ins; i+=2, x++) { ^ src/datasets.c:107:17: note: Uninitialized variable: hash memcpy(out, hash, outs); ^ Bug: #5291. --- src/datasets.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/datasets.c b/src/datasets.c index 3964622d51..c339a28a54 100644 --- a/src/datasets.c +++ b/src/datasets.c @@ -83,12 +83,15 @@ static Dataset *DatasetSearchByName(const char *name) static int HexToRaw(const uint8_t *in, size_t ins, uint8_t *out, size_t outs) { + if (ins < 2) + return -1; if (ins % 2 != 0) return -1; if (outs != ins / 2) return -1; uint8_t hash[outs]; + memset(hash, 0, outs); size_t i, x; for (x = 0, i = 0; i < ins; i+=2, x++) { char buf[3] = { 0, 0, 0 };