advapi32/tests: Don't test function directly when reporting GetLastError().
authorAndré Hentschel <nerv@dawncrow.de>
Sun, 2 Jan 2011 17:48:08 +0000 (18:48 +0100)
committerAlexandre Julliard <julliard@winehq.org>
Mon, 3 Jan 2011 10:54:06 +0000 (11:54 +0100)
dlls/advapi32/tests/security.c

index ff07513f9fe4ba1febfdbf0d4bfb5445db7ce3c5..58107528054364c3ab8f677a3301a235d4ab2025 100644 (file)
@@ -2318,9 +2318,9 @@ static void test_process_security_child(void)
     ok(handle != NULL, "OpenProcess(PROCESS_TERMINATE) with err:%d\n", GetLastError());
     TEST_GRANTED_ACCESS( handle, PROCESS_TERMINATE );
 
-    ok(DuplicateHandle( GetCurrentProcess(), handle, GetCurrentProcess(),
-                        &handle1, 0, TRUE, DUPLICATE_SAME_ACCESS ),
-       "duplicating handle err:%d\n", GetLastError());
+    ret = DuplicateHandle( GetCurrentProcess(), handle, GetCurrentProcess(),
+                           &handle1, 0, TRUE, DUPLICATE_SAME_ACCESS );
+    ok(ret, "duplicating handle err:%d\n", GetLastError());
     TEST_GRANTED_ACCESS( handle1, PROCESS_TERMINATE );
 
     CloseHandle( handle1 );
@@ -2344,23 +2344,23 @@ static void test_process_security_child(void)
     ok(handle == NULL, "OpenProcess(PROCESS_ALL_ACCESS) should have failed\n");
 
     /* Documented privilege elevation */
-    ok(DuplicateHandle( GetCurrentProcess(), GetCurrentProcess(), GetCurrentProcess(),
-                        &handle, 0, TRUE, DUPLICATE_SAME_ACCESS ),
-       "duplicating handle err:%d\n", GetLastError());
+    ret = DuplicateHandle( GetCurrentProcess(), GetCurrentProcess(), GetCurrentProcess(),
+                           &handle, 0, TRUE, DUPLICATE_SAME_ACCESS );
+    ok(ret, "duplicating handle err:%d\n", GetLastError());
     TEST_GRANTED_ACCESS2( handle, PROCESS_ALL_ACCESS,
                           STANDARD_RIGHTS_ALL | SPECIFIC_RIGHTS_ALL );
 
     CloseHandle( handle );
 
     /* Same only explicitly asking for all access rights */
-    ok(DuplicateHandle( GetCurrentProcess(), GetCurrentProcess(), GetCurrentProcess(),
-                        &handle, PROCESS_ALL_ACCESS, TRUE, 0 ),
-       "duplicating handle err:%d\n", GetLastError());
+    ret = DuplicateHandle( GetCurrentProcess(), GetCurrentProcess(), GetCurrentProcess(),
+                           &handle, PROCESS_ALL_ACCESS, TRUE, 0 );
+    ok(ret, "duplicating handle err:%d\n", GetLastError());
     TEST_GRANTED_ACCESS2( handle, PROCESS_ALL_ACCESS,
                           PROCESS_ALL_ACCESS | PROCESS_QUERY_LIMITED_INFORMATION );
-    ok(DuplicateHandle( GetCurrentProcess(), handle, GetCurrentProcess(),
-                        &handle1, PROCESS_VM_READ, TRUE, 0 ),
-       "duplicating handle err:%d\n", GetLastError());
+    ret = DuplicateHandle( GetCurrentProcess(), handle, GetCurrentProcess(),
+                           &handle1, PROCESS_VM_READ, TRUE, 0 );
+    ok(ret, "duplicating handle err:%d\n", GetLastError());
     TEST_GRANTED_ACCESS( handle1, PROCESS_VM_READ );
     CloseHandle( handle1 );
     CloseHandle( handle );