[PATCH 3/4] mf/tests: Always initialize sample data in create_sample helper.

Rémi Bernon rbernon at codeweavers.com
Tue Feb 22 09:47:20 CST 2022


Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=45988
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=47084
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=49715
Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=52183
Signed-off-by: Rémi Bernon <rbernon at codeweavers.com>
---
 dlls/mf/tests/mf.c | 21 +++++++++------------
 1 file changed, 9 insertions(+), 12 deletions(-)

diff --git a/dlls/mf/tests/mf.c b/dlls/mf/tests/mf.c
index c61a8079163..70e2d07f4ca 100644
--- a/dlls/mf/tests/mf.c
+++ b/dlls/mf/tests/mf.c
@@ -5481,18 +5481,15 @@ static IMFSample *create_sample(const BYTE *data, ULONG size)
     ok(hr == S_OK, "MFCreateSample returned %#x\n", hr);
     hr = MFCreateMemoryBuffer(size, &media_buffer);
     ok(hr == S_OK, "MFCreateMemoryBuffer returned %#x\n", hr);
-    if (data)
-    {
-        hr = IMFMediaBuffer_SetCurrentLength(media_buffer, size);
-        ok(hr == S_OK, "SetCurrentLength returned %#x\n", hr);
-        hr = IMFMediaBuffer_Lock(media_buffer, &buffer, NULL, &length);
-        ok(hr == S_OK, "Lock returned %#x\n", hr);
-        ok(length == size, "got size %u\n", length);
-        if (!data) memset(buffer, 0xcd, length);
-        else memcpy(buffer, data, length);
-        hr = IMFMediaBuffer_Unlock(media_buffer);
-        ok(hr == S_OK, "Unlock returned %#x\n", hr);
-    }
+    hr = IMFMediaBuffer_Lock(media_buffer, &buffer, NULL, &length);
+    ok(hr == S_OK, "Lock returned %#x\n", hr);
+    ok(length == 0, "got length %u\n", length);
+    if (!data) memset(buffer, 0xcd, size);
+    else memcpy(buffer, data, size);
+    hr = IMFMediaBuffer_Unlock(media_buffer);
+    ok(hr == S_OK, "Unlock returned %#x\n", hr);
+    hr = IMFMediaBuffer_SetCurrentLength(media_buffer, data ? size : 0);
+    ok(hr == S_OK, "SetCurrentLength returned %#x\n", hr);
     hr = IMFSample_AddBuffer(sample, media_buffer);
     ok(hr == S_OK, "AddBuffer returned %#x\n", hr);
     ret = IMFMediaBuffer_Release(media_buffer);
-- 
2.34.1




More information about the wine-devel mailing list