tests: Don't call memcmp() with NULL arguments
You're not supposed to pass NULL to memcmp(), and some sanitizers complain
about it, even when the length is zero.
Signed-off-by: David Gibson <[email protected]>
diff --git a/tests/testutils.c b/tests/testutils.c
index 3c83b96..378869a 100644
--- a/tests/testutils.c
+++ b/tests/testutils.c
@@ -127,7 +127,7 @@
if (proplen != len)
FAIL("Size mismatch on property \"%s\": %d insead of %d",
name, proplen, len);
- if (memcmp(val, prop->data, len) != 0)
+ if (len && memcmp(val, prop->data, len) != 0)
FAIL("Data mismatch on property \"%s\"", name);
}
@@ -144,7 +144,7 @@
if (proplen != len)
FAIL("Size mismatch on property \"%s\": %d insead of %d",
name, proplen, len);
- if (memcmp(val, propval, len) != 0)
+ if (len && memcmp(val, propval, len) != 0)
FAIL("Data mismatch on property \"%s\"", name);
return propval;
