advapi32/tests: make sure to use return value (LLVM/Clang) (2/2)

Austin English austinenglish at gmail.com
Thu Feb 3 20:28:47 CST 2011


-- 
-Austin
-------------- next part --------------
diff --git a/dlls/advapi32/tests/crypt_lmhash.c b/dlls/advapi32/tests/crypt_lmhash.c
index 384221f..398a47a 100644
--- a/dlls/advapi32/tests/crypt_lmhash.c
+++ b/dlls/advapi32/tests/crypt_lmhash.c
@@ -211,6 +211,7 @@ static void test_SystemFunction003(void)
 
     memset(output, 0, sizeof output);
     r = pSystemFunction002(data, key, output);
+    ok(r == STATUS_SUCCESS, "function failed\n");
 
     ok( !memcmp(exp2, output, sizeof output), "decrypted message wrong\n");
 }
@@ -284,16 +285,19 @@ static void test_SystemFunction004(void)
 
     memset(output, 0, sizeof output);
     r = pSystemFunction002(out.Buffer, key.Buffer, output);
+    ok(r == STATUS_SUCCESS, "function failed\n");
 
     ok(((unsigned int*)output)[0] == in.Length, "crypted length wrong\n");
     ok(((unsigned int*)output)[1] == 1, "crypted value wrong\n");
 
     memset(output, 0, sizeof output);
     r = pSystemFunction002(out.Buffer+8, key.Buffer, output);
+    ok(r == STATUS_SUCCESS, "function failed\n");
     ok(!memcmp(output, inbuf, sizeof output), "crypted data wrong\n");
 
     memset(output, 0, sizeof output);
     r = pSystemFunction002(out.Buffer+16, key.Buffer, output);
+    ok(r == STATUS_SUCCESS, "function failed\n");
     ok(!memcmp(output, inbuf, sizeof output), "crypted data wrong\n");
 }
 
@@ -500,6 +504,7 @@ static void test_memcmpfunc(memcmpfunc fn)
     {
         /* crashes */
         r = fn(NULL, NULL);
+	ok( r == 1, "wrong error code\n");
     }
 
     memset(arg1, 0, sizeof arg1);


More information about the wine-patches mailing list