Nikolay Sivov : mf/samplegrabber: Use OnSetPresentationClock() on shutdown as well.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 729ed419ae4e8083a1a188b6dce08a20d13c5abd
URL:    https://gitlab.winehq.org/wine/wine/-/commit/729ed419ae4e8083a1a188b6dce08a2...

Author: Nikolay Sivov 
Date:   Tue Oct 31 13:04:01 2023 +0100

mf/samplegrabber: Use OnSetPresentationClock() on shutdown as well.

Signed-off-by: Nikolay Sivov

---

dlls/mf/samplegrabber.c | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/dlls/mf/samplegrabber.c b/dlls/mf/samplegrabber.c
index 66ac7808cc7..de599139736 100644
--- a/dlls/mf/samplegrabber.c
+++ b/dlls/mf/samplegrabber.c
@@ -1019,8 +1019,13 @@ static void sample_grabber_cancel_timer(struct sample_grabber *grabber)
     }
 }

-static void sample_grabber_set_presentation_clock(struct sample_grabber *grabber, IMFPresentationClock *clock)
+static HRESULT sample_grabber_set_presentation_clock(struct sample_grabber *grabber, IMFPresentationClock *clock)
 {
+    HRESULT hr;
+
+    if (FAILED(hr = IMFSampleGrabberSinkCallback_OnSetPresentationClock(sample_grabber_get_callback(grabber), clock)))
+        return hr;
+
     if (grabber->clock)
     {
         sample_grabber_cancel_timer(grabber);
@@ -1043,6 +1048,8 @@ static void sample_grabber_set_presentation_clock(struct sample_grabber *grabber
             grabber->timer = NULL;
         }
     }
+
+    return hr;
 }

static HRESULT WINAPI sample_grabber_sink_SetPresentationClock(IMFMediaSink *iface, IMFPresentationClock *clock)
@@ -1058,10 +1065,9 @@ static HRESULT WINAPI sample_grabber_sink_SetPresentationClock(IMFMediaSink *ifa
     {
         hr = MF_E_SHUTDOWN;
     }
-    else if (SUCCEEDED(hr = IMFSampleGrabberSinkCallback_OnSetPresentationClock(sample_grabber_get_callback(grabber),
-            clock)))
+    else
     {
-        sample_grabber_set_presentation_clock(grabber, clock);
+        hr = sample_grabber_set_presentation_clock(grabber, clock);
     }

LeaveCriticalSection(&grabber->cs);
@@ -1109,8 +1115,9 @@ static HRESULT WINAPI sample_grabber_sink_Shutdown(IMFMediaSink *iface)
     {
         grabber->is_shut_down = TRUE;
         sample_grabber_release_pending_items(grabber);
-        sample_grabber_set_presentation_clock(grabber, NULL);
-        if (SUCCEEDED(hr = IMFSampleGrabberSinkCallback_OnShutdown(sample_grabber_get_callback(grabber))))
+        if (SUCCEEDED(hr = sample_grabber_set_presentation_clock(grabber, NULL)))
+            hr = IMFSampleGrabberSinkCallback_OnShutdown(sample_grabber_get_callback(grabber));
+        if (SUCCEEDED(hr))
         {
             IMFMediaType_Release(grabber->current_media_type);
             grabber->current_media_type = NULL;