[PATCH] mf/evr: Remove duplicated check in GetStreamSinkCount() (Coverity).
Nikolay Sivov
nsivov at codeweavers.com
Thu Sep 16 04:58:49 CDT 2021
Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
---
dlls/mf/evr.c | 3 ---
dlls/mf/tests/mf.c | 6 ++++++
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/dlls/mf/evr.c b/dlls/mf/evr.c
index a5545c157de..9552990006d 100644
--- a/dlls/mf/evr.c
+++ b/dlls/mf/evr.c
@@ -1291,9 +1291,6 @@ static HRESULT WINAPI video_renderer_sink_GetStreamSinkCount(IMFMediaSink *iface
TRACE("%p, %p.\n", iface, count);
- if (!count)
- return E_POINTER;
-
EnterCriticalSection(&renderer->cs);
if (renderer->flags & EVR_SHUT_DOWN)
hr = MF_E_SHUTDOWN;
diff --git a/dlls/mf/tests/mf.c b/dlls/mf/tests/mf.c
index 3dfd852643c..1740e4e36d9 100644
--- a/dlls/mf/tests/mf.c
+++ b/dlls/mf/tests/mf.c
@@ -4559,6 +4559,12 @@ todo_wine
hr = IMFMediaSink_Shutdown(sink);
ok(hr == S_OK, "Unexpected hr %#x.\n", hr);
+ hr = IMFMediaSink_GetStreamSinkCount(sink, NULL);
+ ok(hr == MF_E_SHUTDOWN, "Unexpected hr %#x.\n", hr);
+
+ hr = IMFMediaSink_GetStreamSinkCount(sink, &count);
+ ok(hr == MF_E_SHUTDOWN, "Unexpected hr %#x.\n", hr);
+
hr = IMFRateSupport_GetSlowestRate(rs, MFRATE_FORWARD, FALSE, &rate);
ok(hr == MF_E_SHUTDOWN, "Unexpected hr %#x.\n", hr);
--
2.33.0
More information about the wine-devel
mailing list