[PATCH v2 1/6] mfplat/tests: Check that buffers are properly aligned.

Nikolay Sivov nsivov at codeweavers.com
Sun Feb 27 08:12:33 CST 2022



On 2/25/22 19:37, Giovanni Mascellani wrote:
> Signed-off-by: Giovanni Mascellani <gmascellani at codeweavers.com>
> ---
>   dlls/mfplat/tests/mfplat.c | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/dlls/mfplat/tests/mfplat.c b/dlls/mfplat/tests/mfplat.c
> index 91c9d971a3b..9470cf27a90 100644
> --- a/dlls/mfplat/tests/mfplat.c
> +++ b/dlls/mfplat/tests/mfplat.c
> @@ -2286,10 +2286,10 @@ static void test_system_memory_buffer(void)
>       IMFMediaBuffer_Release(buffer);
>   
>       /* Aligned buffer. */
> -    hr = MFCreateAlignedMemoryBuffer(16, MF_8_BYTE_ALIGNMENT, NULL);
> +    hr = MFCreateAlignedMemoryBuffer(16, MF_512_BYTE_ALIGNMENT, NULL);
>       ok(FAILED(hr), "Unexpected hr %#x.\n", hr);
Why was this call changed?
>   
> -    hr = MFCreateAlignedMemoryBuffer(201, MF_8_BYTE_ALIGNMENT, &buffer);
> +    hr = MFCreateAlignedMemoryBuffer(201, MF_512_BYTE_ALIGNMENT, &buffer);
>       ok(hr == S_OK, "Failed to create memory buffer, hr %#x.\n", hr);
>   
>       hr = IMFMediaBuffer_GetCurrentLength(buffer, &length);
> @@ -2317,6 +2317,8 @@ static void test_system_memory_buffer(void)
>       hr = IMFMediaBuffer_Lock(buffer, &data, &max, &length);
>       ok(hr == S_OK, "Failed to lock, hr %#x.\n", hr);
>       ok(max == 201 && length == 10, "Unexpected length.\n");
> +    todo_wine
> +    ok(((uintptr_t)data & MF_512_BYTE_ALIGNMENT) == 0, "Data at %p is insufficiently aligned.\n", data);
>       hr = IMFMediaBuffer_Unlock(buffer);
>       ok(hr == S_OK, "Failed to unlock, hr %#x.\n", hr);
>   




More information about the wine-devel mailing list