qedit/mediadet: Check for failure from IEnumPins::Next().

Fixes a test crash when gstreamer is not present. Signed-off-by: Zebediah Figura <z.figura12@gmail.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

qedit/mediadet: Check for failure from IEnumPins::Next().
9defa529 · Zebediah Figura · Alexandre Julliard · b796b9fb · 9defa529
Commit 9defa529 authored Oct 31, 2019 by Zebediah Figura Committed by Alexandre Julliard Nov 01, 2019
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 1 deletion

mediadet.c dlls/qedit/mediadet.c +6 -1

No files found.
--- a/dlls/qedit/mediadet.c
+++ b/dlls/qedit/mediadet.c
@@ -456,7 +456,12 @@ static HRESULT GetSplitter(MediaDetImpl *This)
            IPin_Release(source_pin);
            goto retry;
        }
-        IEnumPins_Next(pins, 1, &splitter_pin, NULL);
+        if (IEnumPins_Next(pins, 1, &splitter_pin, NULL) != S_OK)
+        {
+            IEnumPins_Release(pins);
+            IPin_Release(source_pin);
+            goto retry;
+        }
        IEnumPins_Release(pins);

        hr = IPin_Connect(source_pin, splitter_pin, NULL);