[PATCH] skip checks if pre condition does not match
Marcus Meissner
marcus at jet.franken.de
Sat Mar 22 13:11:33 CDT 2008
Hi,
spotted by Coverity, we could get negative return values
(or other bad ones) into memcpy().
Just skip the tests if len is not the expected one.
Ciao, Marcus
---
dlls/user32/tests/resource.c | 4 ++++
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/dlls/user32/tests/resource.c b/dlls/user32/tests/resource.c
index 99fc24f..7190e5c 100644
--- a/dlls/user32/tests/resource.c
+++ b/dlls/user32/tests/resource.c
@@ -91,6 +91,10 @@ static void test_LoadStringA (void)
ok (len == expected, "bufsiz=%d: got %d, expected %d\n",
bufsiz, len, expected);
+ if (len != expected) {
+ skip("skipping memcmp and null termination check.\n");
+ continue;
+ }
ok (!memcmp (buf, str, len),
"bufsiz=%d: got '%s', expected '%.*s'\n",
bufsiz, buf, len, str);
--
1.5.2.4
More information about the wine-patches
mailing list