[PATCH v2 0/1] MR2380: quartz: Set filter sync source in FilterGraph2_AddFilter.

[Shaun Ren (＠shaunren)]

This fixes a bug in the game "Nioh: The Complete Edition", where the videos after the first video are played too quickly.

--
  v2: quartz: Set filter sync source in FilterGraph2_AddFilter.

https://gitlab.winehq.org/wine/wine/-/merge_requests/2380

[PATCH v2 1/1] quartz: Set filter sync source in FilterGraph2_AddFilter.

[Shaun Ren]

From: Shaun Ren

---
 dlls/quartz/filtergraph.c       | 2 ++
 dlls/quartz/tests/filtergraph.c | 8 +++++++-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/dlls/quartz/filtergraph.c b/dlls/quartz/filtergraph.c
index 437691b5553..8c8f89ce2ac 100644
--- a/dlls/quartz/filtergraph.c
+++ b/dlls/quartz/filtergraph.c
@@ -663,6 +663,8 @@ static HRESULT WINAPI FilterGraph2_AddFilter(IFilterGraph2 *iface,
         return hr;
     }

+    IBaseFilter_SetSyncSource(filter, graph->refClock);
+
     IBaseFilter_AddRef(entry->filter = filter);

list_add_head(&graph->filters, &entry->entry);
diff --git a/dlls/quartz/tests/filtergraph.c b/dlls/quartz/tests/filtergraph.c
index d6ce9d31ce2..391f9777175 100644
--- a/dlls/quartz/tests/filtergraph.c
+++ b/dlls/quartz/tests/filtergraph.c
@@ -3314,7 +3314,7 @@ static void test_connect_direct(void)

static void test_sync_source(void)
 {
-    struct testfilter filter1, filter2;
+    struct testfilter filter1, filter2, filter3;

IFilterGraph2 *graph = create_graph();
     IReferenceClock *systemclock, *clock;
@@ -3326,6 +3326,7 @@ static void test_sync_source(void)

testfilter_init(&filter1, NULL, 0);
     testfilter_init(&filter2, NULL, 0);
+    testfilter_init(&filter3, NULL, 0);

IFilterGraph2_AddFilter(graph, &filter1.IBaseFilter_iface, NULL);
     IFilterGraph2_AddFilter(graph, &filter2.IBaseFilter_iface, NULL);
@@ -3346,10 +3347,14 @@ static void test_sync_source(void)
     ok(clock == systemclock, "Got clock %p.\n", clock);
     IReferenceClock_Release(clock);

+    IFilterGraph2_AddFilter(graph, &filter3.IBaseFilter_iface, NULL);
+    ok(filter3.clock == systemclock, "Got clock %p.\n", filter3.clock);
+
     hr = IMediaFilter_SetSyncSource(filter, NULL);
     ok(hr == S_OK, "Got hr %#lx.\n", hr);
     ok(!filter1.clock, "Got clock %p.\n", filter1.clock);
     ok(!filter2.clock, "Got clock %p.\n", filter2.clock);
+    ok(!filter3.clock, "Got clock %p.\n", filter3.clock);

hr = IMediaFilter_GetSyncSource(filter, &clock);
     todo_wine ok(hr == S_FALSE, "Got hr %#lx.\n", hr);
@@ -3361,6 +3366,7 @@ static void test_sync_source(void)
     ok(!ref, "Got outstanding refcount %ld\n", ref);
     ok(filter1.ref == 1, "Got outstanding refcount %ld.\n", filter1.ref);
     ok(filter2.ref == 1, "Got outstanding refcount %ld.\n", filter2.ref);
+    ok(filter3.ref == 1, "Got outstanding refcount %ld.\n", filter3.ref);
 }

#define check_filter_state(a, b) check_filter_state_(__LINE__, a, b)

[Shaun Ren (＠shaunren)]

On Sat Mar 18 03:18:17 2023 +0000, Zebediah Figura wrote:
> This patch will leave the filter in the graph but not tracked if
> SetSyncSource() fails. I don't think it's worth putting too much effort
> into testing the error handling, but currently it looks wrong, which
> would be confusing to the reader.

Fixed.