[PATCH v4 0/1] MR7838: mfplat/tests: Don't assume video processor MFT can provide samples.

[Charlotte Pabst (＠CharlottePabst)]

This fixes a test failure/crash on Windows 8.1 introduced by dd1d82728811c15716eefa0917eb3ecac4b9a85a.

--
  v4: mfplat/tests: Don't assume video processor MFT can provide samples.
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7838

[PATCH v4 1/1] mfplat/tests: Don't assume video processor MFT can provide samples.

[Charlotte Pabst]

From: Charlotte Pabst <cpabst(a)codeweavers.com>

This fixes a test failure/crash on Windows 8.1 introduced by dd1d82728811c15716eefa0917eb3ecac4b9a85a.
---
 dlls/mfplat/tests/mfplat.c | 24 +++++++++++++++---------
 1 file changed, 15 insertions(+), 9 deletions(-)

diff --git a/dlls/mfplat/tests/mfplat.c b/dlls/mfplat/tests/mfplat.c
index 8c0b50a85cf..6fe3f40cd93 100644
--- a/dlls/mfplat/tests/mfplat.c
+++ b/dlls/mfplat/tests/mfplat.c
@@ -6828,18 +6828,10 @@ static IMFSample *test_xvp_process_sample(IMFTransform *xvp, IMFSample *in_sampl
         .dwStatus = 0,
         .pEvents = NULL,
     };
-    MFT_OUTPUT_STREAM_INFO info;
     DWORD status;
     BYTE *out_data;
     DWORD out_length;
 
-    hr = IMFTransform_GetOutputStreamInfo(xvp, 0, &info);
-    ok(hr == S_OK, "Failed to get output stream info, hr %#lx.\n", hr);
-
-    ok(!!(info.dwFlags & MFT_OUTPUT_STREAM_PROVIDES_SAMPLES), "MFT_OUTPUT_STREAM_PROVIDES_SAMPLES expected\n");
-    ok(!(info.dwFlags & MFT_OUTPUT_STREAM_CAN_PROVIDE_SAMPLES), "MFT_OUTPUT_STREAM_CAN_PROVIDE_SAMPLES unexpected\n");
-    ok(info.cbSize == 4, "Output size should be 4.\n");
-
     hr = IMFTransform_ProcessInput(xvp, 0, in_sample, 0);
     ok(hr == S_OK, "Failed to process input, hr %#lx.\n", hr);
 
@@ -6869,6 +6861,7 @@ static void test_xvp_playback_mode(void)
 {
     HRESULT hr;
     UINT reset_token;
+    MFT_OUTPUT_STREAM_INFO out_info;
     IMFDXGIDeviceManager *manager;
     ID3D11Device *device;
     IMFTransform *xvp;
@@ -6934,9 +6927,22 @@ static void test_xvp_playback_mode(void)
     hr = IMFTransform_SetOutputType(xvp, 0, type, 0);
     ok(hr == S_OK, "Failed to set output type, hr %#lx.\n", hr);
 
+    hr = IMFTransform_GetOutputStreamInfo(xvp, 0, &out_info);
+    ok(hr == S_OK, "Failed to get output stream info, hr %#lx.\n", hr);
+
+    ok(out_info.cbSize == 4, "Output size should be 4.\n");
+
     in_sample = create_dxgi_sample(device, 0xdeadbeef, 0);
 
-    IMFSample_Release(test_xvp_process_sample(xvp, in_sample, NULL));
+    if ((out_info.dwFlags & MFT_OUTPUT_STREAM_PROVIDES_SAMPLES) || (out_info.dwFlags & MFT_OUTPUT_STREAM_CAN_PROVIDE_SAMPLES))
+    {
+        got_out_sample = test_xvp_process_sample(xvp, in_sample, NULL);
+        IMFSample_Release(got_out_sample);
+    }
+    else
+    {
+        win_skip("Video processor MFT can't provide output samples.\n");
+    }
 
     hr = IMFTransform_ProcessMessage(xvp, MFT_MESSAGE_NOTIFY_END_STREAMING, 0);
     ok(hr == S_OK, "Failed to end streaming, hr %#lx.\n", hr);
-- 
GitLab

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

[Charlotte Pabst (＠CharlottePabst)]

On Mon Apr 21 14:25:50 2025 +0000, Charlotte Pabst wrote:
> changed this line in [version 4 of the diff](/wine/wine/-/merge_requests/7838/diffs?diff_id=172366&start_sha=58ef62b9fc0a8fadb0ab68bbe99029f024de1d22#856f958c3f51ca87a2b3b1d1a0efd2130d1c6c4c_6939_6939)
> done

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7838#note_101559

[PATCH v4 0/1] MR7838: mfplat/tests: Don't assume video processor MFT can provide samples. - approved

[Nikolay Sivov (＠nsivov)]

This merge request was approved by Nikolay Sivov.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7838