[PATCH 0/3] MR6986: mf: Fix a regression in unexpected source shutdown handling.
From: Conor McCarthy <cmccarthy(a)codeweavers.com> --- dlls/mf/session.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/dlls/mf/session.c b/dlls/mf/session.c index cb45fceb714..fa6938ca193 100644 --- a/dlls/mf/session.c +++ b/dlls/mf/session.c @@ -1992,7 +1992,8 @@ static HRESULT session_set_current_topology(struct media_session *session, IMFTo session->source_shutdown_handled = FALSE; - session_collect_nodes(session); + if (FAILED(hr = session_collect_nodes(session))) + return hr; LIST_FOR_EACH_ENTRY(node, &session->presentation.nodes, struct topo_node, entry) { -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/6986
From: Conor McCarthy <cmccarthy(a)codeweavers.com> The status must remain MF_TOPOSTATUS_INVALID because the topology is invalid and must be handled as such in session_start(). Fixes a regression introduced by efb369f5755067829df2aafe9b270d59cf5090ac. --- dlls/mf/session.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/mf/session.c b/dlls/mf/session.c index fa6938ca193..5b711f7712d 100644 --- a/dlls/mf/session.c +++ b/dlls/mf/session.c @@ -516,7 +516,7 @@ static void session_set_topo_status(struct media_session *session, HRESULT statu IMFMediaEvent *event; PROPVARIANT param; - if (topo_status == MF_TOPOSTATUS_INVALID) + if (topo_status == MF_TOPOSTATUS_INVALID || status == MF_E_SHUTDOWN) return; if (list_empty(&session->topologies)) -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/6986
From: Conor McCarthy <cmccarthy(a)codeweavers.com> This test would hang if added before the regression fix. --- dlls/mf/tests/mf.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/dlls/mf/tests/mf.c b/dlls/mf/tests/mf.c index 73526802b63..faf82f676db 100644 --- a/dlls/mf/tests/mf.c +++ b/dlls/mf/tests/mf.c @@ -6453,6 +6453,7 @@ static void test_media_session_source_shutdown(void) HRESULT hr; enum { + TEST_TOPOLOGY, TEST_START, TEST_RESTART, TEST_PAUSE, @@ -6465,7 +6466,7 @@ static void test_media_session_source_shutdown(void) /* These tests don't cover asynchronous shutdown, which is difficult to consistently test. */ - for (shutdown_point = TEST_START; shutdown_point <= TEST_CLOSE; ++shutdown_point) + for (shutdown_point = TEST_TOPOLOGY; shutdown_point <= TEST_CLOSE; ++shutdown_point) { winetest_push_context("Test %d", shutdown_point); @@ -6489,6 +6490,8 @@ static void test_media_session_source_shutdown(void) hr = MFCreateMediaSession(NULL, &session); ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); topology = create_test_topology(source, sink_activate, &duration); + if (shutdown_point == TEST_TOPOLOGY) + IMFMediaSource_Shutdown(source); hr = IMFMediaSession_SetTopology(session, 0, topology); ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); IMFTopology_Release(topology); @@ -6501,7 +6504,7 @@ static void test_media_session_source_shutdown(void) IMFMediaSource_Shutdown(source); ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); hr = wait_media_event(session, callback, MESessionStarted, 5000, &propvar); - ok(hr == (shutdown_point == TEST_START ? MF_E_INVALIDREQUEST : S_OK), "Unexpected hr %#lx.\n", hr); + ok(hr == (shutdown_point <= TEST_START ? MF_E_INVALIDREQUEST : S_OK), "Unexpected hr %#lx.\n", hr); switch (shutdown_point) { -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/6986
Hi, It looks like your patch introduced the new failures shown below. Please investigate and fix them before resubmitting your patch. If they are not new, fixing them anyway would help a lot. Otherwise please ask for the known failures list to be updated. The tests also ran into some preexisting test failures. If you know how to fix them that would be helpful. See the TestBot job for the details: The full results can be found at: https://testbot.winehq.org/JobDetails.pl?Key=150307 Your paranoid android. === w7u_2qxl (32 bit report) === mf: mf.c:5383: Test failed: got hr 0x80072f8f mf.c:5383: Test failed: got hr 0x80072f8f mf.c:5454: Test failed: got hr 0x80072f8f mf.c:5460: Test failed: got hr 0x80072f8f mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf.c:6339: Test failed: WaitForSingleObject returned 258 mf.c:6339: Test failed: Unexpected hr 0xd36d8. mf: Timeout === debian11 (build log) === 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. === debian11b (64 bit WoW report) === dinput: joystick8.c:5762: Test failed: input 1: WaitForSingleObject returned 0x102 joystick8.c:5763: Test failed: input 1: got 0 WM_INPUT messages joystick8.c:5766: Test failed: input 1: got dwType 0 joystick8.c:5767: Test failed: input 1: got header.dwSize 0 joystick8.c:5769: Test failed: input 1: got hDevice 0000000000000000 joystick8.c:5771: Test failed: input 1: got dwSizeHid 0 joystick8.c:5772: Test failed: input 1: got dwCount 0 === debian11b (build log) === 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins. 0138:err:winediag:video_processor_create GStreamer doesn't support video conversion, please install appropriate plugins.
participants (3)
-
Conor McCarthy -
Conor McCarthy (＠cmccarthy)
-
Marvin