Nikolay Sivov : quartz/tests: Add a couple of tests for return values ( Coverity).
Alexandre Julliard
julliard at winehq.org
Mon Nov 13 16:24:19 CST 2017
Module: wine
Branch: master
Commit: b356673d01c29b9cd283b42e86b78588e5ea4ee1
URL: http://source.winehq.org/git/wine.git/?a=commit;h=b356673d01c29b9cd283b42e86b78588e5ea4ee1
Author: Nikolay Sivov <nsivov at codeweavers.com>
Date: Mon Nov 13 21:56:39 2017 +0300
quartz/tests: Add a couple of tests for return values (Coverity).
Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
dlls/quartz/tests/avisplitter.c | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/dlls/quartz/tests/avisplitter.c b/dlls/quartz/tests/avisplitter.c
index 23167b5..8bafb14 100644
--- a/dlls/quartz/tests/avisplitter.c
+++ b/dlls/quartz/tests/avisplitter.c
@@ -422,8 +422,10 @@ static void test_filter_graph(void)
}
ok(hr == S_OK, "Could not create null renderer: %08x\n", hr);
- IBaseFilter_EnumPins(pnull, &nullenum);
- IEnumPins_Next(nullenum, 1, &nullpin, NULL);
+ hr = IBaseFilter_EnumPins(pnull, &nullenum);
+ ok(hr == S_OK, "Failed to enum pins, hr %#x.\n", hr);
+ hr = IEnumPins_Next(nullenum, 1, &nullpin, NULL);
+ ok(hr == S_OK, "Failed to get next pin, hr %#x.\n", hr);
IEnumPins_Release(nullenum);
IPin_QueryDirection(nullpin, &dir);
@@ -489,7 +491,9 @@ fail2:
if (dir == PINDIR_OUTPUT)
{
PIN_INFO info;
- IPin_QueryPinInfo(to, &info);
+
+ hr = IPin_QueryPinInfo(to, &info);
+ ok(hr == S_OK, "Failed to query pin info, hr %#x.\n", hr);
/* Release twice: Once normal, second from the
* previous while loop
More information about the wine-cvs
mailing list