Nikolay Sivov : ole32/tests: Add some return value checks (Coverity).

Alexandre Julliard julliard at winehq.org
Fri Jan 3 15:25:06 CST 2020


Module: wine
Branch: master
Commit: d024b33ca1abffd88f1af07e8c117f529f86190c
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=d024b33ca1abffd88f1af07e8c117f529f86190c

Author: Nikolay Sivov <nsivov at codeweavers.com>
Date:   Fri Jan  3 22:43:42 2020 +0300

ole32/tests: Add some return value checks (Coverity).

Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/ole32/tests/clipboard.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/dlls/ole32/tests/clipboard.c b/dlls/ole32/tests/clipboard.c
index b97c2646bf..0a087aff84 100644
--- a/dlls/ole32/tests/clipboard.c
+++ b/dlls/ole32/tests/clipboard.c
@@ -1000,7 +1000,8 @@ static void test_set_clipboard(void)
 
     test_cf_dataobject(NULL);
 
-    ok(OleSetClipboard(NULL) == S_OK, "failed to clear clipboard, hr = 0x%08x\n", hr);
+    hr = OleSetClipboard(NULL);
+    ok(hr == S_OK, "Failed to clear clipboard, hr = 0x%08x\n", hr);
 
     OpenClipboard(NULL);
     h = GetClipboardData(cf_onemore);
@@ -1180,7 +1181,8 @@ static void test_consumer_refs(void)
 
     ok(get1 != get2, "data objects match\n");
 
-    OleSetClipboard(NULL);
+    hr = OleSetClipboard(NULL);
+    ok(hr == S_OK, "Failed to clear clipboard, hr %#x.\n", hr);
 
     hr = OleGetClipboard(&get3);
     ok(hr == S_OK, "got %08x\n", hr);
@@ -1260,7 +1262,8 @@ static void test_consumer_refs(void)
     refs = count_refs(src2);
     ok(refs == old_refs + 1, "%d %d\n", refs, old_refs);
 
-    OleSetClipboard(NULL);
+    hr = OleSetClipboard(NULL);
+    ok(hr == S_OK, "Failed to clear clipboard, hr %#x.\n", hr);
 
     refs = count_refs(src2);
     ok(refs == 2, "%d\n", refs);
@@ -1274,7 +1277,8 @@ static void test_consumer_refs(void)
     old_refs = count_refs(src);
     ok(old_refs == 1, "%d\n", old_refs);
 
-    OleSetClipboard(src);
+    hr = OleSetClipboard(src);
+    ok(hr == S_OK, "Failed to clear clipboard, hr %#x.\n", hr);
     refs = count_refs(src);
     ok(refs > old_refs, "%d %d\n", refs, old_refs);
 




More information about the wine-cvs mailing list