Dmitry Timoshkov : psapi: Add missing return value checks.

Alexandre Julliard julliard at winehq.org
Tue Mar 27 12:48:07 CDT 2012


Module: wine
Branch: master
Commit: 4c6a19837b7eb2477a65ab107d36405446d8c28b
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=4c6a19837b7eb2477a65ab107d36405446d8c28b

Author: Dmitry Timoshkov <dmitry at baikal.ru>
Date:   Thu Mar 22 16:27:26 2012 +0800

psapi: Add missing return value checks.

---

 dlls/psapi/tests/psapi_main.c |   22 +++++++++++++++-------
 1 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/dlls/psapi/tests/psapi_main.c b/dlls/psapi/tests/psapi_main.c
index 2087881..1944065 100644
--- a/dlls/psapi/tests/psapi_main.c
+++ b/dlls/psapi/tests/psapi_main.c
@@ -178,15 +178,20 @@ static void test_GetProcessMemoryInfo(void)
     DWORD ret;
 
     SetLastError(0xdeadbeef);
-    pGetProcessMemoryInfo(NULL, &pmc, sizeof(pmc));
+    ret = pGetProcessMemoryInfo(NULL, &pmc, sizeof(pmc));
+    ok(!ret, "GetProcessMemoryInfo should fail\n");
     ok(GetLastError() == ERROR_INVALID_HANDLE, "expected error=ERROR_INVALID_HANDLE but got %d\n", GetLastError());
 
     SetLastError(0xdeadbeef);
-    pGetProcessMemoryInfo(hpSR, &pmc, sizeof(pmc));
-    todo_wine ok(GetLastError() == ERROR_ACCESS_DENIED, "expected error=ERROR_ACCESS_DENIED but got %d\n", GetLastError());
+    ret = pGetProcessMemoryInfo(hpSR, &pmc, sizeof(pmc));
+todo_wine
+    ok(!ret, "GetProcessMemoryInfo should fail\n");
+todo_wine
+    ok(GetLastError() == ERROR_ACCESS_DENIED, "expected error=ERROR_ACCESS_DENIED but got %d\n", GetLastError());
 
     SetLastError(0xdeadbeef);
-    pGetProcessMemoryInfo(hpQI, &pmc, sizeof(pmc)-1);
+    ret = pGetProcessMemoryInfo(hpQI, &pmc, sizeof(pmc)-1);
+    ok(!ret, "GetProcessMemoryInfo should fail\n");
     ok(GetLastError() == ERROR_INSUFFICIENT_BUFFER, "expected error=ERROR_INSUFFICIENT_BUFFER but got %d\n", GetLastError());
 
     SetLastError(0xdeadbeef);
@@ -239,13 +244,16 @@ static void test_GetMappedFileName(void)
     HANDLE hfile, hmap;
 
     SetLastError(0xdeadbeef);
-    pGetMappedFileNameA(NULL, hMod, szMapPath, sizeof(szMapPath));
+    ret = pGetMappedFileNameA(NULL, hMod, szMapPath, sizeof(szMapPath));
+    ok(!ret, "GetMappedFileName should fail\n");
 todo_wine
     ok(GetLastError() == ERROR_INVALID_HANDLE, "expected error=ERROR_INVALID_HANDLE but got %d\n", GetLastError());
 
     SetLastError(0xdeadbeef);
-    pGetMappedFileNameA(hpSR, hMod, szMapPath, sizeof(szMapPath));
-    todo_wine ok(GetLastError() == ERROR_ACCESS_DENIED, "expected error=ERROR_ACCESS_DENIED but got %d\n", GetLastError());
+    ret = pGetMappedFileNameA(hpSR, hMod, szMapPath, sizeof(szMapPath));
+    ok(!ret, "GetMappedFileName should fail\n");
+todo_wine
+    ok(GetLastError() == ERROR_ACCESS_DENIED, "expected error=ERROR_ACCESS_DENIED but got %d\n", GetLastError());
 
     SetLastError( 0xdeadbeef );
     ret = pGetMappedFileNameA(hpQI, hMod, szMapPath, sizeof(szMapPath));




More information about the wine-cvs mailing list