[PATCH] amstream/tests: Add return value checks (Coverity).

Nikolay Sivov nsivov at codeweavers.com
Mon Nov 9 08:08:03 CST 2020


Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
---

For consistency with other tests.

 dlls/amstream/tests/amstream.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/dlls/amstream/tests/amstream.c b/dlls/amstream/tests/amstream.c
index da5ef8f56a1..b12c205db55 100644
--- a/dlls/amstream/tests/amstream.c
+++ b/dlls/amstream/tests/amstream.c
@@ -496,7 +496,8 @@ static void test_openfile(const WCHAR *test_avi_path)
     ok(hr == S_OK, "Got hr %#x.\n", hr);
 
     clock = NULL;
-    IMediaFilter_GetSyncSource(media_filter, &clock);
+    hr = IMediaFilter_GetSyncSource(media_filter, &clock);
+    ok(hr == S_OK, "Got hr %#x.\n", hr);
     ok(!!clock, "Expected non-NULL clock.\n");
 
     hr = IAMMultiMediaStream_SetState(mmstream, STREAMSTATE_STOP);
@@ -551,7 +552,8 @@ static void test_openfile(const WCHAR *test_avi_path)
     ok(hr == S_OK, "Got hr %#x.\n", hr);
 
     clock = (IReferenceClock *)0xdeadbeef;
-    IMediaFilter_GetSyncSource(media_filter, &clock);
+    hr = IMediaFilter_GetSyncSource(media_filter, &clock);
+    ok(hr == S_OK, "Got hr %#x.\n", hr);
     ok(!clock, "Got clock %p.\n", clock);
 
     hr = IAMMultiMediaStream_SetState(mmstream, STREAMSTATE_STOP);
-- 
2.28.0




More information about the wine-devel mailing list