Nikolay Sivov : comctl32/tests: Test some return values (Coverity).

Alexandre Julliard julliard at winehq.org
Thu Dec 28 15:18:28 CST 2017


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

Author: Nikolay Sivov <nsivov at codeweavers.com>
Date:   Thu Dec 28 15:20:17 2017 +0300

comctl32/tests: Test some return values (Coverity).

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

---

 dlls/comctl32/tests/imagelist.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/dlls/comctl32/tests/imagelist.c b/dlls/comctl32/tests/imagelist.c
index e0251a9..d72e22a 100644
--- a/dlls/comctl32/tests/imagelist.c
+++ b/dlls/comctl32/tests/imagelist.c
@@ -836,8 +836,11 @@ static const IStreamVtbl Test_Stream_Vtbl =
 
 static void init_memstream(struct memstream *stream)
 {
+    HRESULT hr;
+
     stream->IStream_iface.lpVtbl = &Test_Stream_Vtbl;
-    CreateStreamOnHGlobal(NULL, TRUE, &stream->stream);
+    hr = CreateStreamOnHGlobal(NULL, TRUE, &stream->stream);
+    ok(hr == S_OK, "Failed to create a stream, hr %#x.\n", hr);
 }
 
 static void cleanup_memstream(struct memstream *stream)
@@ -995,7 +998,8 @@ static void check_iml_data(HIMAGELIST himl, INT cx, INT cy, INT cur, INT max, IN
     hr = GetHGlobalFromStream(stream.stream, &hglobal);
     ok(hr == S_OK, "%s: Failed to get hglobal, %#x\n", comment, hr);
 
-    IStream_Stat(stream.stream, &stat, STATFLAG_NONAME);
+    hr = IStream_Stat(stream.stream, &stat, STATFLAG_NONAME);
+    ok(hr == S_OK, "Stat() failed, hr %#x.\n", hr);
 
     data = GlobalLock(hglobal);
 




More information about the wine-cvs mailing list