February 2023 - Wine-commits

Hans Leidekker : winscard: Improve traces.
by Alexandre Julliard 22 Feb '23

22 Feb '23

Module: wine Branch: master Commit: 21b4de5444435bda7b832c7cfb71c99290bd08fe URL: https://gitlab.winehq.org/wine/wine/-/commit/21b4de5444435bda7b832c7cfb71c9… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Thu Feb 16 13:33:49 2023 +0100 winscard: Improve traces. --- dlls/winscard/winscard.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/dlls/winscard/winscard.c b/dlls/winscard/winscard.c index b2dd7d5f8e9..c7954faa400 100644 --- a/dlls/winscard/winscard.c +++ b/dlls/winscard/winscard.c @@ -112,6 +112,7 @@ static inline char *utf8_to_ansi( const char *src ) HANDLE WINAPI SCardAccessStartedEvent(void) { + FIXME( "stub\n" ); return g_startedEvent; } @@ -185,9 +186,10 @@ LONG WINAPI SCardIsValidContext( SCARDCONTEXT context ) return ret; } -LONG WINAPI SCardListCardsA(SCARDCONTEXT hContext, LPCBYTE pbAtr, LPCGUID rgguidInterfaces, DWORD cguidInterfaceCount, LPSTR mszCards, LPDWORD pcchCards) +LONG WINAPI SCardListCardsA( SCARDCONTEXT context, const BYTE *atr, const GUID *interfaces, DWORD interface_count, + char *cards, DWORD *cards_len ) { - FIXME(": stub\n"); + FIXME( "%Ix, %p, %p, %lu, %p, %p stub\n", context, atr, interfaces, interface_count, cards, cards_len ); SetLastError(ERROR_CALL_NOT_IMPLEMENTED); return SCARD_F_INTERNAL_ERROR; } @@ -343,7 +345,7 @@ LONG WINAPI SCardStatusW( SCARDHANDLE connect, WCHAR *names, DWORD *names_len, D void WINAPI SCardReleaseStartedEvent(void) { - FIXME("stub\n"); + FIXME( "stub\n" ); } LONG WINAPI SCardListReadersA( SCARDCONTEXT context, const char *groups, char *readers, DWORD *readers_len )

1 0

Hans Leidekker : winscard: Add wow64 thunks.
by Alexandre Julliard 22 Feb '23

22 Feb '23

Module: wine Branch: master Commit: 190a31d1abf99661d17d839c90305ece0c3ee3f1 URL: https://gitlab.winehq.org/wine/wine/-/commit/190a31d1abf99661d17d839c90305e… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Thu Feb 16 13:36:55 2023 +0100 winscard: Add wow64 thunks. --- dlls/winscard/unixlib.c | 353 ++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 353 insertions(+) diff --git a/dlls/winscard/unixlib.c b/dlls/winscard/unixlib.c index 4fb17b8ad6f..cd55e6352a8 100644 --- a/dlls/winscard/unixlib.c +++ b/dlls/winscard/unixlib.c @@ -176,3 +176,356 @@ const unixlib_entry_t __wine_unix_call_funcs[] = scard_get_attrib, scard_set_attrib, }; + +#ifdef _WIN64 + +typedef ULONG PTR32; + +static NTSTATUS wow64_scard_establish_context( void *args ) +{ + struct + { + UINT64 scope; + PTR32 handle; + } const *params32 = args; + + struct scard_establish_context_params params = + { + params32->scope, + ULongToPtr(params32->handle) + }; + return scard_establish_context( &params ); +} + +static NTSTATUS wow64_scard_release_context( void *args ) +{ + struct + { + UINT64 handle; + } const *params32 = args; + + struct scard_release_context_params params = + { + params32->handle + }; + return scard_release_context( &params ); +} + +static NTSTATUS wow64_scard_is_valid_context( void *args ) +{ + struct + { + UINT64 handle; + } const *params32 = args; + + struct scard_is_valid_context_params params = + { + params32->handle + }; + return scard_is_valid_context( &params ); +} + +static NTSTATUS wow64_scard_get_status_change( void *args ) +{ + struct + { + UINT64 handle; + UINT64 timeout; + PTR32 states; + UINT64 count; + } const *params32 = args; + + struct scard_get_status_change_params params = + { + params32->handle, + params32->timeout, + ULongToPtr(params32->states), + params32->count, + }; + return scard_get_status_change( &params ); +} + +static NTSTATUS wow64_scard_cancel( void *args ) +{ + struct + { + UINT64 handle; + } const *params32 = args; + + struct scard_cancel_params params = + { + params32->handle + }; + return scard_cancel( &params ); +} + +static NTSTATUS wow64_scard_list_readers( void *args ) +{ + struct + { + UINT64 handle; + PTR32 groups; + PTR32 readers; + PTR32 readers_len; + } *params32 = args; + + struct scard_list_readers_params params = + { + params32->handle, + ULongToPtr(params32->groups), + ULongToPtr(params32->readers), + ULongToPtr(params32->readers_len) + }; + return scard_list_readers( &params ); +}; + +static NTSTATUS wow64_scard_list_reader_groups( void *args ) +{ + struct + { + UINT64 handle; + PTR32 groups; + PTR32 groups_len; + } *params32 = args; + + struct scard_list_reader_groups_params params = + { + params32->handle, + ULongToPtr(params32->groups), + ULongToPtr(params32->groups_len) + }; + return scard_list_reader_groups( &params ); +} + +static NTSTATUS wow64_scard_connect( void *args ) +{ + struct + { + UINT64 context_handle; + PTR32 reader; + UINT64 share_mode; + UINT64 preferred_protocols; + PTR32 connect_handle; + PTR32 protocol; + } *params32 = args; + + struct scard_connect_params params = + { + params32->context_handle, + ULongToPtr(params32->reader), + params32->share_mode, + params32->preferred_protocols, + ULongToPtr(params32->connect_handle), + ULongToPtr(params32->protocol) + }; + return scard_connect( &params ); +} + +static NTSTATUS wow64_scard_status( void *args ) +{ + struct + { + UINT64 handle; + PTR32 names; + PTR32 names_len; + PTR32 state; + PTR32 protocol; + PTR32 atr; + PTR32 atr_len; + } *params32 = args; + + struct scard_status_params params = + { + params32->handle, + ULongToPtr(params32->names), + ULongToPtr(params32->names_len), + ULongToPtr(params32->state), + ULongToPtr(params32->protocol), + ULongToPtr(params32->atr), + ULongToPtr(params32->atr_len) + }; + return scard_status( &params ); +}; + +static NTSTATUS wow64_scard_reconnect( void *args ) +{ + struct + { + UINT64 handle; + UINT64 share_mode; + UINT64 preferred_protocols; + UINT64 initialization; + PTR32 protocol; + } *params32 = args; + + struct scard_reconnect_params params = + { + params32->handle, + params32->share_mode, + params32->preferred_protocols, + params32->initialization, + ULongToPtr(params32->protocol) + }; + return scard_reconnect( &params ); +} + +static NTSTATUS wow64_scard_disconnect( void *args ) +{ + struct + { + UINT64 handle; + UINT64 disposition; + } *params32 = args; + + struct scard_disconnect_params params = + { + params32->handle, + params32->disposition + }; + return scard_disconnect( &params ); +} + +static NTSTATUS wow64_scard_begin_transaction( void *args ) +{ + struct + { + UINT64 handle; + } *params32 = args; + + struct scard_begin_transaction_params params = + { + params32->handle + }; + return scard_begin_transaction( &params ); +} + +static NTSTATUS wow64_scard_end_transaction( void *args ) +{ + struct + { + UINT64 handle; + UINT64 disposition; + } *params32 = args; + + struct scard_end_transaction_params params = + { + params32->handle, + params32->disposition + }; + return scard_end_transaction( &params ); +} + +static NTSTATUS wow64_scard_transmit( void *args ) +{ + struct + { + UINT64 handle; + PTR32 send; + PTR32 send_buf; + UINT64 send_buflen; + PTR32 recv; + PTR32 recv_buf; + PTR32 recv_buflen; + } *params32 = args; + + struct scard_transmit_params params = + { + params32->handle, + ULongToPtr(params32->send), + ULongToPtr(params32->send_buf), + params32->send_buflen, + ULongToPtr(params32->recv), + ULongToPtr(params32->recv_buf), + ULongToPtr(params32->recv_buflen) + }; + return scard_transmit( &params ); +}; + +static NTSTATUS wow64_scard_control( void *args ) +{ + struct + { + UINT64 handle; + UINT64 code; + PTR32 in_buf; + UINT64 in_buflen; + PTR32 out_buf; + UINT64 out_buflen; + PTR32 ret_len; + } *params32 = args; + + struct scard_control_params params = + { + params32->handle, + params32->code, + ULongToPtr(params32->in_buf), + params32->in_buflen, + ULongToPtr(params32->out_buf), + params32->out_buflen, + ULongToPtr(params32->ret_len) + }; + return scard_control( &params ); +}; + +static NTSTATUS wow64_scard_get_attrib( void *args ) +{ + struct + { + UINT64 handle; + UINT64 id; + PTR32 attr; + PTR32 attr_len; + } *params32 = args; + + struct scard_get_attrib_params params = + { + params32->handle, + params32->id, + ULongToPtr(params32->attr), + ULongToPtr(params32->attr_len) + }; + return scard_get_attrib( &params ); +} + +static NTSTATUS wow64_scard_set_attrib( void *args ) +{ + struct + { + UINT64 handle; + UINT64 id; + PTR32 attr; + UINT64 attr_len; + } *params32 = args; + + struct scard_set_attrib_params params = + { + params32->handle, + params32->id, + ULongToPtr(params32->attr), + params32->attr_len + }; + return scard_set_attrib( &params ); +} + +const unixlib_entry_t __wine_unix_call_wow64_funcs[] = +{ + wow64_scard_establish_context, + wow64_scard_release_context, + wow64_scard_is_valid_context, + wow64_scard_get_status_change, + wow64_scard_cancel, + wow64_scard_list_readers, + wow64_scard_list_reader_groups, + wow64_scard_connect, + wow64_scard_status, + wow64_scard_reconnect, + wow64_scard_disconnect, + wow64_scard_begin_transaction, + wow64_scard_end_transaction, + wow64_scard_transmit, + wow64_scard_control, + wow64_scard_get_attrib, + wow64_scard_set_attrib, +}; + +#endif /* _WIN64 */

1 0

Hans Leidekker : winscard/tests: Initial tests.
by Alexandre Julliard 22 Feb '23

22 Feb '23

Module: wine Branch: master Commit: 77fdfdb8ef09153bf0adbfa6226b6477b4dd0702 URL: https://gitlab.winehq.org/wine/wine/-/commit/77fdfdb8ef09153bf0adbfa6226b64… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Thu Feb 16 13:44:26 2023 +0100 winscard/tests: Initial tests. --- configure | 1 + configure.ac | 1 + dlls/winscard/tests/Makefile.in | 5 + dlls/winscard/tests/winscard.c | 237 ++++++++++++++++++++++++++++++++++++++++ 4 files changed, 244 insertions(+)

1 0

Francois Gouget : vbscript/tests: Fix the testChrError() tests.
by Alexandre Julliard 22 Feb '23

22 Feb '23

Module: wine Branch: master Commit: 15b176b4f4945d7abfb4adbddc7f140ba1765855 URL: https://gitlab.winehq.org/wine/wine/-/commit/15b176b4f4945d7abfb4adbddc7f14… Author: Francois Gouget <fgouget(a)codeweavers.com> Date: Tue Feb 21 20:08:29 2023 +0100 vbscript/tests: Fix the testChrError() tests. Chr() depends on the thread's ANSI code page which in some cases is different from the Windows ANSI code page (e.g. hi-IN). Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54003 --- dlls/vbscript/tests/run.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/vbscript/tests/run.c b/dlls/vbscript/tests/run.c index 4158aa28197..294b635ea41 100644 --- a/dlls/vbscript/tests/run.c +++ b/dlls/vbscript/tests/run.c @@ -239,7 +239,7 @@ static void detect_locale(void) PRIMARYLANGID(GetUserDefaultUILanguage()) == LANG_ENGLISH && PRIMARYLANGID(GetUserDefaultLangID()) == LANG_ENGLISH); - GetCPInfoExA( CP_ACP, 0, &cpinfo ); + GetCPInfoExA( CP_THREAD_ACP, 0, &cpinfo ); MaxCharSize = cpinfo.MaxCharSize; GetLocaleInfoA(LOCALE_USER_DEFAULT, LOCALE_IFIRSTDAYOFWEEK | LOCALE_RETURN_NUMBER,

1 0

Ziqing Hui : mf/tests: Add missing \n.
by Alexandre Julliard 22 Feb '23

22 Feb '23

Module: wine Branch: master Commit: a5b5cef355568a59dd8633fc6a608d9f68598df8 URL: https://gitlab.winehq.org/wine/wine/-/commit/a5b5cef355568a59dd8633fc6a608d… Author: Ziqing Hui <zhui(a)codeweavers.com> Date: Thu Feb 9 17:40:25 2023 +0800 mf/tests: Add missing \n. --- dlls/mf/tests/transform.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/dlls/mf/tests/transform.c b/dlls/mf/tests/transform.c index f806485df78..0c9f7548d42 100644 --- a/dlls/mf/tests/transform.c +++ b/dlls/mf/tests/transform.c @@ -1323,11 +1323,11 @@ static void check_dmo_set_input_type(IMediaObject *media_object, const GUID *sub { flag = flags[i]; hr = IMediaObject_SetInputType(media_object, 1, NULL, flag); - ok(hr == DMO_E_INVALIDSTREAMINDEX, "SetInputType returned %#lx for flag %#lx.", hr, flag); + ok(hr == DMO_E_INVALIDSTREAMINDEX, "SetInputType returned %#lx for flag %#lx.\n", hr, flag); hr = IMediaObject_SetInputType(media_object, 1, &bad_media_type, flag); - ok(hr == DMO_E_INVALIDSTREAMINDEX, "SetInputType returned %#lx for flag %#lx.", hr, flag); + ok(hr == DMO_E_INVALIDSTREAMINDEX, "SetInputType returned %#lx for flag %#lx.\n", hr, flag); hr = IMediaObject_SetInputType(media_object, 1, good_media_type, flag); - ok(hr == DMO_E_INVALIDSTREAMINDEX, "SetInputType returned %#lx for flag %#lx.", hr, flag); + ok(hr == DMO_E_INVALIDSTREAMINDEX, "SetInputType returned %#lx for flag %#lx.\n", hr, flag); } /* Test unaccepted type. */ @@ -1337,10 +1337,10 @@ static void check_dmo_set_input_type(IMediaObject *media_object, const GUID *sub if (!(flag & DMO_SET_TYPEF_CLEAR)) { hr = IMediaObject_SetInputType(media_object, 0, NULL, flag); - ok(hr == DMO_E_TYPE_NOT_ACCEPTED, "SetInputType returned %#lx for flag %#lx.", hr, flag); + ok(hr == DMO_E_TYPE_NOT_ACCEPTED, "SetInputType returned %#lx for flag %#lx.\n", hr, flag); } hr = IMediaObject_SetInputType(media_object, 0, &bad_media_type, flag); - ok(hr == DMO_E_TYPE_NOT_ACCEPTED, "SetInputType returned %#lx for flag %#lx.", hr, flag); + ok(hr == DMO_E_TYPE_NOT_ACCEPTED, "SetInputType returned %#lx for flag %#lx.\n", hr, flag); } /* Test clearing the type. */ @@ -1348,7 +1348,7 @@ static void check_dmo_set_input_type(IMediaObject *media_object, const GUID *sub { flag = DMO_SET_TYPEF_CLEAR | flags[i]; hr = IMediaObject_SetInputType(media_object, 0, NULL, flag); - ok(hr == S_OK, "SetInputType returned %#lx for flag %#lx.", hr, flag); + ok(hr == S_OK, "SetInputType returned %#lx for flag %#lx.\n", hr, flag); } /* Test accepted type. */ @@ -1356,7 +1356,7 @@ static void check_dmo_set_input_type(IMediaObject *media_object, const GUID *sub { flag = flags[i]; hr = IMediaObject_SetInputType(media_object, 0, good_media_type, flag); - ok(hr == S_OK, "SetInputType returned %#lx for flag %#lx.", hr, flag); + ok(hr == S_OK, "SetInputType returned %#lx for flag %#lx.\n", hr, flag); } /* Test unconsidered header member.*/

1 0

Ziqing Hui : mf/tests: Correctly initialize DMO media type.
by Alexandre Julliard 22 Feb '23

22 Feb '23

Module: wine Branch: master Commit: 41288d0d701f7933538c3e20055f0e6b23935a59 URL: https://gitlab.winehq.org/wine/wine/-/commit/41288d0d701f7933538c3e20055f0e… Author: Ziqing Hui <zhui(a)codeweavers.com> Date: Wed Feb 22 11:13:29 2023 +0800 mf/tests: Correctly initialize DMO media type. --- dlls/mf/tests/transform.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/dlls/mf/tests/transform.c b/dlls/mf/tests/transform.c index 0c9f7548d42..7ce68728574 100644 --- a/dlls/mf/tests/transform.c +++ b/dlls/mf/tests/transform.c @@ -327,11 +327,14 @@ static void init_dmo_media_type_video(DMO_MEDIA_TYPE *media_type, { VIDEOINFOHEADER *header = (VIDEOINFOHEADER *)(media_type + 1); BOOL compressed = is_compressed_subtype(subtype); + ULONG codec_data_size = compressed ? 4 : 0; + + memset(media_type, 0, sizeof(*media_type) + sizeof(*header) + codec_data_size); - memset(header, 0, sizeof(*header)); header->bmiHeader.biSize = sizeof(header->bmiHeader); header->bmiHeader.biWidth = width; header->bmiHeader.biHeight = height; + header->bmiHeader.biPlanes = 1; header->bmiHeader.biBitCount = subtype_to_bpp(subtype); header->bmiHeader.biCompression = subtype_to_compression(subtype); @@ -342,7 +345,7 @@ static void init_dmo_media_type_video(DMO_MEDIA_TYPE *media_type, media_type->lSampleSize = 0; media_type->formattype = FORMAT_VideoInfo; media_type->pUnk = NULL; - media_type->cbFormat = sizeof(*header) + 4; /* 4 bytes codec data. */ + media_type->cbFormat = sizeof(*header) + codec_data_size; media_type->pbFormat = (BYTE *)header; }

1 0

Ziqing Hui : mf/tests: Test ProcessInput and ProcessOutput for WMV decoder DMO.
by Alexandre Julliard 22 Feb '23

22 Feb '23

Module: wine Branch: master Commit: 9d1b927924062a8485928d0d53709b6af292d637 URL: https://gitlab.winehq.org/wine/wine/-/commit/9d1b927924062a8485928d0d53709b… Author: Ziqing Hui <zhui(a)codeweavers.com> Date: Mon Jan 30 09:46:07 2023 +0800 mf/tests: Test ProcessInput and ProcessOutput for WMV decoder DMO. --- dlls/mf/tests/transform.c | 136 +++++++++++++++++++++++++++++++++++++++++++++- 1 file changed, 134 insertions(+), 2 deletions(-) diff --git a/dlls/mf/tests/transform.c b/dlls/mf/tests/transform.c index 7ce68728574..72440051192 100644 --- a/dlls/mf/tests/transform.c +++ b/dlls/mf/tests/transform.c @@ -55,6 +55,97 @@ DEFINE_GUID(MFVideoFormat_WMV_Unknown,0x7ce12ca9,0xbfbf,0x43d9,0x9d,0x00,0x82,0x DEFINE_GUID(mft_output_sample_incomplete,0xffffff,0xffff,0xffff,0xff,0xff,0xff,0xff,0xff,0xff,0xff,0xff); +struct media_buffer +{ + IMediaBuffer IMediaBuffer_iface; + LONG refcount; + DWORD length; + DWORD max_length; + BYTE data[]; +}; + +static inline struct media_buffer *impl_from_IMediaBuffer(IMediaBuffer *iface) +{ + return CONTAINING_RECORD(iface, struct media_buffer, IMediaBuffer_iface); +} + +static HRESULT WINAPI media_buffer_QueryInterface(IMediaBuffer *iface, REFIID iid, void **obj) +{ + if (IsEqualIID(iid, &IID_IMediaBuffer) + || IsEqualIID(iid, &IID_IUnknown)) + { + *obj = iface; + return S_OK; + } + return E_NOINTERFACE; +} + +static ULONG WINAPI media_buffer_AddRef(IMediaBuffer *iface) +{ + struct media_buffer *buffer = impl_from_IMediaBuffer(iface); + return InterlockedIncrement(&buffer->refcount); +} + +static ULONG WINAPI media_buffer_Release(IMediaBuffer *iface) +{ + struct media_buffer *buffer = impl_from_IMediaBuffer(iface); + ULONG ref = InterlockedDecrement(&buffer->refcount); + if (!ref) + free(buffer); + return ref; +} + +static HRESULT WINAPI media_buffer_SetLength(IMediaBuffer *iface, DWORD length) +{ + struct media_buffer *buffer = impl_from_IMediaBuffer(iface); + if (length > buffer->max_length) + return E_INVALIDARG; + buffer->length = length; + return S_OK; +} + +static HRESULT WINAPI media_buffer_GetMaxLength(IMediaBuffer *iface, DWORD *max_length) +{ + struct media_buffer *buffer = impl_from_IMediaBuffer(iface); + if (!max_length) + return E_POINTER; + *max_length = buffer->max_length; + return S_OK; +} + +static HRESULT WINAPI media_buffer_GetBufferAndLength(IMediaBuffer *iface, BYTE **data, DWORD *length) +{ + struct media_buffer *buffer = impl_from_IMediaBuffer(iface); + if (!data || ! length) + return E_POINTER; + *data = buffer->data; + *length = buffer->length; + return S_OK; +} + +static IMediaBufferVtbl media_buffer_vtbl = { + media_buffer_QueryInterface, + media_buffer_AddRef, + media_buffer_Release, + media_buffer_SetLength, + media_buffer_GetMaxLength, + media_buffer_GetBufferAndLength, +}; + +HRESULT media_buffer_create(DWORD max_length, struct media_buffer **ret) +{ + struct media_buffer *buffer; + + if (!(buffer = calloc(1, offsetof(struct media_buffer, data[max_length])))) + return E_OUTOFMEMORY; + buffer->IMediaBuffer_iface.lpVtbl = &media_buffer_vtbl; + buffer->refcount = 1; + buffer->length = 0; + buffer->max_length = max_length; + *ret = buffer; + return S_OK; +} + static BOOL is_compressed_subtype(const GUID *subtype) { if (IsEqualGUID(subtype, &MEDIASUBTYPE_WMV1) @@ -1526,7 +1617,6 @@ static void check_dmo_set_output_type(IMediaObject *media_object, const GUID *su } } - static HRESULT WINAPI test_unk_QueryInterface(IUnknown *iface, REFIID riid, void **obj) { if (IsEqualIID(riid, &IID_IUnknown)) @@ -5078,10 +5168,14 @@ static void test_wmv_decoder_media_object(void) {MFMediaType_Video, MEDIASUBTYPE_RGB555, TRUE, FALSE, 512, FORMAT_VideoInfo, NULL, 88, (BYTE *)&expected_output_info[11]}, {MFMediaType_Video, MEDIASUBTYPE_RGB8, TRUE, FALSE, 256, FORMAT_VideoInfo, NULL, 1112, (BYTE *)&expected_output_info[12]}, }; + const DWORD data_width = 96, data_height = 96; const POINT test_size[] = {{16, 16}, {96, 96}, {320, 240}}; - DWORD in_count, out_count, size, expected_size, alignment; + DWORD in_count, out_count, size, expected_size, alignment, wmv_data_length, status; + struct media_buffer *input_media_buffer = NULL, *output_media_buffer = NULL; + DMO_OUTPUT_DATA_BUFFER output_data_buffer; DMO_MEDIA_TYPE media_type, *type; IMediaObject *media_object; + const BYTE *wmv_data; char buffer[1024]; ULONG ret, i, j; HRESULT hr; @@ -5205,6 +5299,44 @@ static void test_wmv_decoder_media_object(void) winetest_pop_context(); } + /* Test ProcessInput. */ + load_resource(L"wmvencdata.bin", &wmv_data, &wmv_data_length); + wmv_data_length = *((DWORD *)wmv_data); + wmv_data += sizeof(DWORD); + hr = media_buffer_create(wmv_data_length, &input_media_buffer); + ok(hr == S_OK, "Failed to create input media buffer.\n"); + memcpy(input_media_buffer->data, wmv_data, wmv_data_length); + input_media_buffer->length = wmv_data_length; + + init_dmo_media_type_video(type, &MEDIASUBTYPE_WMV1, data_width, data_height); + hr = IMediaObject_SetInputType(media_object, 0, type, 0); + ok(hr == S_OK, "SetInputType returned %#lx.\n", hr); + init_dmo_media_type_video(type, &MEDIASUBTYPE_NV12, data_width, data_height); + hr = IMediaObject_SetOutputType(media_object, 0, type, 0); + ok(hr == S_OK, "SetOutputType returned %#lx.\n", hr); + + hr = IMediaObject_ProcessInput(media_object, 0, &input_media_buffer->IMediaBuffer_iface, 0, 0, 0); + todo_wine + ok(hr == S_OK, "ProcessInput returned %#lx.\n", hr); + + /* Test ProcessOutput. */ + hr = IMediaObject_GetOutputSizeInfo(media_object, 0, &size, &alignment); + ok(hr == S_OK, "GetOutputSizeInfo returned %#lx.\n", hr); + hr = media_buffer_create(size, &output_media_buffer); + ok(hr == S_OK, "Failed to create output media buffer.\n"); + output_data_buffer.pBuffer = &output_media_buffer->IMediaBuffer_iface; + output_data_buffer.dwStatus = 0xdeadbeef; + output_data_buffer.rtTimestamp = 0xdeadbeef; + output_data_buffer.rtTimelength = 0xdeadbeef; + hr = IMediaObject_ProcessOutput(media_object, 0, 1, &output_data_buffer, &status); + todo_wine + ok(hr == S_OK, "ProcessOutput returned %#lx.\n", hr); + + ret = IMediaBuffer_Release(&output_media_buffer->IMediaBuffer_iface); + ok(ret == 0, "Release returned %lu\n", ret); + ret = IMediaBuffer_Release(&input_media_buffer->IMediaBuffer_iface); + ok(ret == 0, "Release returned %lu\n", ret); + ret = IMediaObject_Release(media_object); ok(ret == 0, "Release returned %lu\n", ret); CoUninitialize();

1 0

Ziqing Hui : mf/tests: Test output data for WMV decoder DMO.
by Alexandre Julliard 22 Feb '23

22 Feb '23

Module: wine Branch: master Commit: 13a65b849e07a7f66cc122efa6a8c9aed4a56dc4 URL: https://gitlab.winehq.org/wine/wine/-/commit/13a65b849e07a7f66cc122efa6a8c9… Author: Ziqing Hui <zhui(a)codeweavers.com> Date: Thu Feb 2 11:13:46 2023 +0800 mf/tests: Test output data for WMV decoder DMO. --- dlls/mf/tests/transform.c | 66 +++++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 64 insertions(+), 2 deletions(-) diff --git a/dlls/mf/tests/transform.c b/dlls/mf/tests/transform.c index 72440051192..7d217dfe8c0 100644 --- a/dlls/mf/tests/transform.c +++ b/dlls/mf/tests/transform.c @@ -1617,6 +1617,50 @@ static void check_dmo_set_output_type(IMediaObject *media_object, const GUID *su } } +#define check_dmo_output_data_buffer(a, b, c) check_dmo_output_data_buffer_(__LINE__, a, b, c) +static DWORD check_dmo_output_data_buffer_(int line, DMO_OUTPUT_DATA_BUFFER *output_data_buffer, + const struct sample_desc *sample_desc, const WCHAR *expect_data_filename) +{ + const struct buffer_desc *buffer_desc = &sample_desc->buffers[0]; + DWORD diff, data_length, buffer_length, expect_length; + BYTE *data, *buffer; + HRESULT hr; + + if (output_data_buffer->dwStatus & DMO_OUTPUT_DATA_BUFFERF_TIME) + ok_(__FILE__, line)(output_data_buffer->rtTimestamp == sample_desc->sample_time, + "Unexpected time %I64d, expected %I64d.\n", + output_data_buffer->rtTimestamp, sample_desc->sample_time); + if (output_data_buffer->dwStatus & DMO_OUTPUT_DATA_BUFFERF_TIMELENGTH) + ok_(__FILE__, line)(output_data_buffer->rtTimelength == sample_desc->sample_duration, + "Unexpected duration %I64d, expected %I64d.\n", + output_data_buffer->rtTimelength, sample_desc->sample_duration); + + load_resource(expect_data_filename, (const BYTE **)&data, &data_length); + + expect_length = buffer_desc->length; + if (expect_length == -1) + { + expect_length = *(DWORD *)data; + data += sizeof(DWORD); + data_length = data_length - sizeof(DWORD); + } + + hr = IMediaBuffer_GetBufferAndLength(output_data_buffer->pBuffer, &buffer, &buffer_length); + ok(hr == S_OK, "GetBufferAndLength returned %#lx.\n", hr); + ok_(__FILE__, line)(buffer_length == expect_length, "Unexpected length %#lx, expected %#lx\n", buffer_length, expect_length); + + diff = 0; + if (data_length < buffer_length) + ok_(__FILE__, line)(0, "Missing %#lx bytes\n", buffer_length - data_length); + else if (!buffer_desc->compare) + diff = compare_bytes(buffer, &buffer_length, NULL, data); + else + diff = buffer_desc->compare(buffer, &buffer_length, &buffer_desc->rect, data); + + return diff; +} + + static HRESULT WINAPI test_unk_QueryInterface(IUnknown *iface, REFIID riid, void **obj) { if (IsEqualIID(riid, &IID_IUnknown)) @@ -5170,7 +5214,17 @@ static void test_wmv_decoder_media_object(void) }; const DWORD data_width = 96, data_height = 96; const POINT test_size[] = {{16, 16}, {96, 96}, {320, 240}}; - DWORD in_count, out_count, size, expected_size, alignment, wmv_data_length, status; + const struct buffer_desc output_buffer_desc_nv12 = + { + .length = data_width * data_height * 3 / 2, + .compare = compare_nv12, .dump = dump_nv12, .rect = {.right = 82, .bottom = 84}, + }; + const struct sample_desc output_sample_desc_nv12 = + { + .sample_time = 0, .sample_duration = 333333, + .buffer_count = 1, .buffers = &output_buffer_desc_nv12, + }; + DWORD in_count, out_count, size, expected_size, alignment, wmv_data_length, status, expected_status, diff; struct media_buffer *input_media_buffer = NULL, *output_media_buffer = NULL; DMO_OUTPUT_DATA_BUFFER output_data_buffer; DMO_MEDIA_TYPE media_type, *type; @@ -5315,7 +5369,7 @@ static void test_wmv_decoder_media_object(void) hr = IMediaObject_SetOutputType(media_object, 0, type, 0); ok(hr == S_OK, "SetOutputType returned %#lx.\n", hr); - hr = IMediaObject_ProcessInput(media_object, 0, &input_media_buffer->IMediaBuffer_iface, 0, 0, 0); + hr = IMediaObject_ProcessInput(media_object, 0, &input_media_buffer->IMediaBuffer_iface, 0, 0, 333333); todo_wine ok(hr == S_OK, "ProcessInput returned %#lx.\n", hr); @@ -5331,6 +5385,14 @@ static void test_wmv_decoder_media_object(void) hr = IMediaObject_ProcessOutput(media_object, 0, 1, &output_data_buffer, &status); todo_wine ok(hr == S_OK, "ProcessOutput returned %#lx.\n", hr); + expected_status = DMO_OUTPUT_DATA_BUFFERF_SYNCPOINT | DMO_OUTPUT_DATA_BUFFERF_TIME | DMO_OUTPUT_DATA_BUFFERF_TIMELENGTH; + todo_wine + ok(output_data_buffer.dwStatus == expected_status, "Got unexpected dwStatus %#lx.\n", output_data_buffer.dwStatus); + if (hr == S_OK) + { + diff = check_dmo_output_data_buffer(&output_data_buffer, &output_sample_desc_nv12, L"nv12frame.bmp"); + ok(diff == 0, "Got %lu%% diff.\n", diff); + } ret = IMediaBuffer_Release(&output_media_buffer->IMediaBuffer_iface); ok(ret == 0, "Release returned %lu\n", ret);

1 0

Paul Gofman : win32u: Expose and use ProcessEvents from drivers instead of MsgWaitForMultipleObjectsEx.
by Alexandre Julliard 22 Feb '23

22 Feb '23

Module: wine Branch: master Commit: a97fd9f29e7dbeee2dc06415a34a9f7669ea2e0d URL: https://gitlab.winehq.org/wine/wine/-/commit/a97fd9f29e7dbeee2dc06415a34a9f… Author: Paul Gofman <pgofman(a)codeweavers.com> Date: Tue Feb 21 14:43:39 2023 -0600 win32u: Expose and use ProcessEvents from drivers instead of MsgWaitForMultipleObjectsEx. --- dlls/win32u/dce.c | 3 +-- dlls/win32u/driver.c | 12 ++++-------- dlls/win32u/input.c | 3 +-- dlls/win32u/message.c | 20 +++++++++++++------- dlls/wineandroid.drv/android.h | 4 +--- dlls/wineandroid.drv/init.c | 2 +- dlls/wineandroid.drv/window.c | 11 ++++------- dlls/winemac.drv/event.c | 27 +++++---------------------- dlls/winemac.drv/gdi.c | 2 +- dlls/winemac.drv/macdrv.h | 4 +--- dlls/winex11.drv/event.c | 25 ++++--------------------- dlls/winex11.drv/init.c | 2 +- dlls/winex11.drv/x11drv.h | 4 +--- include/wine/gdi_driver.h | 2 +- 14 files changed, 39 insertions(+), 82 deletions(-)

1 0

Alexandre Julliard : makedep: Don't add dependencies for tests of disabled dlls.
by Alexandre Julliard 22 Feb '23

22 Feb '23

Module: wine Branch: master Commit: 8670876db9b7b918cb7893a16d7edde6b40a607a URL: https://gitlab.winehq.org/wine/wine/-/commit/8670876db9b7b918cb7893a16d7edd… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Feb 22 21:04:35 2023 +0100 makedep: Don't add dependencies for tests of disabled dlls. --- tools/makedep.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/tools/makedep.c b/tools/makedep.c index 532145de292..b94a221854d 100644 --- a/tools/makedep.c +++ b/tools/makedep.c @@ -3558,6 +3558,11 @@ static void output_test_module( struct makefile *make, unsigned int arch ) output( "\t%secho \"%s_test.exe TESTRES \\\"%s\\\"\" | %s -u -o $@\n", cmd_prefix( "WRC" ), basemodule, obj_dir_path( make, stripped ), tools_path( make, "wrc" )); + if (make->disabled[arch] || (parent && parent->disabled[arch])) + { + make->ok_files = empty_strarray; + return; + } output_filenames_obj_dir( make, make->ok_files ); output( ": %s", obj_dir_path( make, testmodule )); if (parent) @@ -3568,8 +3573,7 @@ static void output_test_module( struct makefile *make, unsigned int arch ) } output( "\n" ); output( "%s %s:", obj_dir_path( make, "check" ), obj_dir_path( make, "test" )); - if (!make->disabled[arch] && parent && !parent->disabled[arch]) - output_filenames_obj_dir( make, make->ok_files ); + output_filenames_obj_dir( make, make->ok_files ); output( "\n" ); strarray_add_uniq( &make->phony_targets, obj_dir_path( make, "check" )); strarray_add_uniq( &make->phony_targets, obj_dir_path( make, "test" ));

1 0