November 2022 - Wine-Commits

Michel Zou : msvcrt: Fix errno set on dup from read-only file.
by Alexandre Julliard Nov. 30, 2022

Nov. 30, 2022

Module: wine Branch: master Commit: 53bc6c7a4a416252559a2a174821001fdd576c6f URL: https://gitlab.winehq.org/wine/wine/-/commit/53bc6c7a4a416252559a2a17482100… Author: Michel Zou <xantares09(a)hotmail.com> Date: Sat Nov 26 18:27:52 2022 +0100 msvcrt: Fix errno set on dup from read-only file. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53950 --- dlls/msvcrt/file.c | 1 + dlls/msvcrt/tests/file.c | 6 ++++++ 2 files changed, 7 insertions(+) diff --git a/dlls/msvcrt/file.c b/dlls/msvcrt/file.c index 4a8c3510bd7..e72784eef41 100644 --- a/dlls/msvcrt/file.c +++ b/dlls/msvcrt/file.c @@ -1244,6 +1244,7 @@ int CDECL _dup2(int od, int nd) if (info_nd == &MSVCRT___badioinfo) { + *_errno() = EBADF; ret = -1; } else if (info_od->wxflag & WX_OPEN) diff --git a/dlls/msvcrt/tests/file.c b/dlls/msvcrt/tests/file.c index de21d11d0b9..3f57b3f871f 100644 --- a/dlls/msvcrt/tests/file.c +++ b/dlls/msvcrt/tests/file.c @@ -267,6 +267,12 @@ static void test_readmode( BOOL ascii_mode ) close (fd); + fd = open ("fdopen.tst", O_RDONLY, _S_IREAD |_S_IWRITE); + errno = 0xdeadbeef; + ok(dup2(fd, -1) == -1, "dup2(fd, -1) succeeded\n"); + ok(errno == EBADF, "errno = %d\n", errno); + close (fd); + if (ascii_mode) { /* Open file in ascii mode */ fd = open ("fdopen.tst", O_RDONLY);

1 0

Rémi Bernon : joy.cpl: Remove unnecessary JoystickData structure.
by Alexandre Julliard Nov. 30, 2022

Nov. 30, 2022

Module: wine Branch: master Commit: 140456a05834999ee2fd612e2aef8d52386ea562 URL: https://gitlab.winehq.org/wine/wine/-/commit/140456a05834999ee2fd612e2aef8d… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue Nov 29 21:41:21 2022 +0100 joy.cpl: Remove unnecessary JoystickData structure. --- dlls/joy.cpl/main.c | 57 ++++++++++++++--------------------------------------- 1 file changed, 15 insertions(+), 42 deletions(-) diff --git a/dlls/joy.cpl/main.c b/dlls/joy.cpl/main.c index 8e8d167068e..28334ec6f99 100644 --- a/dlls/joy.cpl/main.c +++ b/dlls/joy.cpl/main.c @@ -44,11 +44,6 @@ struct device IDirectInputDevice8W *device; }; -struct JoystickData -{ - IDirectInput8W *di; -}; - static HMODULE hcpl; static CRITICAL_SECTION joy_cs; @@ -81,12 +76,12 @@ BOOL WINAPI DllMain(HINSTANCE hdll, DWORD reason, LPVOID reserved) static BOOL CALLBACK enum_devices( const DIDEVICEINSTANCEW *instance, void *context ) { DIDEVCAPS caps = {.dwSize = sizeof(DIDEVCAPS)}; - struct JoystickData *data = context; + IDirectInput8W *dinput = context; struct device *entry; if (!(entry = calloc( 1, sizeof(*entry) ))) return DIENUM_STOP; - IDirectInput8_CreateDevice( data->di, &instance->guidInstance, &entry->device, NULL ); + IDirectInput8_CreateDevice( dinput, &instance->guidInstance, &entry->device, NULL ); IDirectInputDevice8_SetDataFormat( entry->device, &c_dfDIJoystick ); IDirectInputDevice8_GetCapabilities( entry->device, &caps ); @@ -170,8 +165,9 @@ static void enable_joystick(WCHAR *joy_name, BOOL enable) if (appkey) RegCloseKey(appkey); } -static void refresh_joystick_list(HWND hwnd, struct JoystickData *data) +static void refresh_joystick_list( HWND hwnd ) { + IDirectInput8W *dinput; struct device *entry; HKEY hkey, appkey; DWORD values = 0; @@ -180,7 +176,9 @@ static void refresh_joystick_list(HWND hwnd, struct JoystickData *data) clear_devices(); - IDirectInput8_EnumDevices( data->di, DI8DEVCLASS_GAMECTRL, enum_devices, data, DIEDFL_ATTACHEDONLY ); + DirectInput8Create( GetModuleHandleW( NULL ), DIRECTINPUT_VERSION, &IID_IDirectInput8W, (void **)&dinput, NULL ); + IDirectInput8_EnumDevices( dinput, DI8DEVCLASS_GAMECTRL, enum_devices, dinput, DIEDFL_ATTACHEDONLY ); + IDirectInput8_Release( dinput ); SendDlgItemMessageW(hwnd, IDC_JOYSTICKLIST, LB_RESETCONTENT, 0, 0); SendDlgItemMessageW(hwnd, IDC_DISABLEDLIST, LB_RESETCONTENT, 0, 0); @@ -247,7 +245,6 @@ static void override_joystick(WCHAR *joy_name, BOOL override) static INT_PTR CALLBACK list_dlgproc(HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam) { WCHAR instance_name[MAX_PATH] = {0}; - static struct JoystickData *data; int sel; TRACE("(%p, 0x%08x/%d, 0x%Ix)\n", hwnd, msg, msg, lparam); @@ -255,9 +252,7 @@ static INT_PTR CALLBACK list_dlgproc(HWND hwnd, UINT msg, WPARAM wparam, LPARAM { case WM_INITDIALOG: { - data = (struct JoystickData*) ((PROPSHEETPAGEW*)lparam)->lParam; - - refresh_joystick_list(hwnd, data); + refresh_joystick_list( hwnd ); EnableWindow(GetDlgItem(hwnd, IDC_BUTTONENABLE), FALSE); EnableWindow(GetDlgItem(hwnd, IDC_BUTTONDISABLE), FALSE); @@ -281,7 +276,7 @@ static INT_PTR CALLBACK list_dlgproc(HWND hwnd, UINT msg, WPARAM wparam, LPARAM if (instance_name[0]) { enable_joystick(instance_name, FALSE); - refresh_joystick_list(hwnd, data); + refresh_joystick_list( hwnd ); } } break; @@ -294,7 +289,7 @@ static INT_PTR CALLBACK list_dlgproc(HWND hwnd, UINT msg, WPARAM wparam, LPARAM if (instance_name[0]) { enable_joystick(instance_name, TRUE); - refresh_joystick_list(hwnd, data); + refresh_joystick_list( hwnd ); } } break; @@ -305,7 +300,7 @@ static INT_PTR CALLBACK list_dlgproc(HWND hwnd, UINT msg, WPARAM wparam, LPARAM { SendDlgItemMessageW(hwnd, IDC_JOYSTICKLIST, LB_GETTEXT, sel, (LPARAM)instance_name); override_joystick(instance_name, FALSE); - refresh_joystick_list(hwnd, data); + refresh_joystick_list( hwnd ); } } break; @@ -316,7 +311,7 @@ static INT_PTR CALLBACK list_dlgproc(HWND hwnd, UINT msg, WPARAM wparam, LPARAM { SendDlgItemMessageW(hwnd, IDC_XINPUTLIST, LB_GETTEXT, sel, (LPARAM)instance_name); override_joystick(instance_name, TRUE); - refresh_joystick_list(hwnd, data); + refresh_joystick_list( hwnd ); } } break; @@ -375,7 +370,7 @@ static int CALLBACK propsheet_callback(HWND hwnd, UINT msg, LPARAM lparam) return 0; } -static void display_cpl_sheets( HWND parent, struct JoystickData *data ) +static void display_cpl_sheets( HWND parent ) { INITCOMMONCONTROLSEX init = { @@ -389,14 +384,12 @@ static void display_cpl_sheets( HWND parent, struct JoystickData *data ) .hInstance = hcpl, .pszTemplate = MAKEINTRESOURCEW( IDD_LIST ), .pfnDlgProc = list_dlgproc, - .lParam = (INT_PTR)data, }, { .dwSize = sizeof(PROPSHEETPAGEW), .hInstance = hcpl, .pszTemplate = MAKEINTRESOURCEW( IDD_TEST_DI ), .pfnDlgProc = test_di_dialog_proc, - .lParam = (INT_PTR)data, }, { .dwSize = sizeof(PROPSHEETPAGEW), @@ -499,30 +492,14 @@ static void unregister_window_class(void) */ LONG CALLBACK CPlApplet(HWND hwnd, UINT command, LPARAM lParam1, LPARAM lParam2) { - static struct JoystickData data; TRACE("(%p, %u, 0x%Ix, 0x%Ix)\n", hwnd, command, lParam1, lParam2); switch (command) { case CPL_INIT: - { - HRESULT hr; - register_window_class(); - - /* Initialize dinput */ - hr = DirectInput8Create(GetModuleHandleW(NULL), DIRECTINPUT_VERSION, &IID_IDirectInput8W, (void**)&data.di, NULL); - - if (FAILED(hr)) - { - ERR("Failed to initialize DirectInput: 0x%08lx\n", hr); - return FALSE; - } - - IDirectInput8_EnumDevices( data.di, DI8DEVCLASS_GAMECTRL, enum_devices, &data, DIEDFL_ATTACHEDONLY ); - return TRUE; - } + case CPL_GETCOUNT: return 1; @@ -538,15 +515,11 @@ LONG CALLBACK CPlApplet(HWND hwnd, UINT command, LPARAM lParam1, LPARAM lParam2) } case CPL_DBLCLK: - display_cpl_sheets(hwnd, &data); + display_cpl_sheets( hwnd ); break; case CPL_STOP: clear_devices(); - - /* And destroy dinput too */ - IDirectInput8_Release(data.di); - unregister_window_class(); break; }

1 0

Rémi Bernon : joy.cpl: Move dinput test tab logic to dinput.c.
by Alexandre Julliard Nov. 30, 2022

Nov. 30, 2022

Module: wine Branch: master Commit: 9d80a0ee790adbbd4bcc9997bbb23aa86e0a1c06 URL: https://gitlab.winehq.org/wine/wine/-/commit/9d80a0ee790adbbd4bcc9997bbb23a… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Tue Nov 29 21:20:41 2022 +0100 joy.cpl: Move dinput test tab logic to dinput.c. --- dlls/joy.cpl/dinput.c | 514 +++++++++++++++++++++++++++++++++++++++++++-- dlls/joy.cpl/joy_private.h | 2 +- dlls/joy.cpl/main.c | 496 +------------------------------------------ 3 files changed, 494 insertions(+), 518 deletions(-)

1 0

Nikolay Sivov : d3dcompiler: Implement D3DDisassemble() using vkd3d-shader.
by Alexandre Julliard Nov. 30, 2022

Nov. 30, 2022

Module: wine Branch: master Commit: 7bf58c895c3145187bbc6dcbe4e0e14d11457a39 URL: https://gitlab.winehq.org/wine/wine/-/commit/7bf58c895c3145187bbc6dcbe4e0e1… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sun Nov 27 09:20:14 2022 +0300 d3dcompiler: Implement D3DDisassemble() using vkd3d-shader. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=46649 Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com> --- dlls/d3dcompiler_43/compiler.c | 74 +++++++++++++++++++++++++++++++++++++++-- dlls/d3dcompiler_43/tests/asm.c | 6 ++-- 2 files changed, 73 insertions(+), 7 deletions(-) diff --git a/dlls/d3dcompiler_43/compiler.c b/dlls/d3dcompiler_43/compiler.c index 174fbc98280..8fb38cf592c 100644 --- a/dlls/d3dcompiler_43/compiler.c +++ b/dlls/d3dcompiler_43/compiler.c @@ -596,11 +596,79 @@ HRESULT WINAPI D3DPreprocess(const void *data, SIZE_T size, const char *filename return preprocess_shader(data, size, filename, defines, include, shader, error_messages); } -HRESULT WINAPI D3DDisassemble(const void *data, SIZE_T size, UINT flags, const char *comments, ID3DBlob **disassembly) +HRESULT WINAPI D3DDisassemble(const void *data, SIZE_T size, UINT flags, const char *comments, + ID3DBlob **disassembly) { - FIXME("data %p, size %Iu, flags %#x, comments %p, disassembly %p stub!\n", + struct vkd3d_shader_compile_info compile_info; + enum vkd3d_shader_source_type source_type; + struct vkd3d_shader_code asm_code; + const char *ptr = data; + char *messages; + HRESULT hr; + int ret; + + TRACE("data %p, size %Iu, flags %#x, comments %p, disassembly %p.\n", data, size, flags, comments, disassembly); - return E_NOTIMPL; + + if (flags) + FIXME("Ignoring flags %#x.\n", flags); + + if (comments) + FIXME("Ignoring comments %s.\n", debugstr_a(comments)); + +#if D3D_COMPILER_VERSION >= 46 + if (!size) + return E_INVALIDARG; +#endif + + if (size >= 4 && read_u32(&ptr) == TAG_DXBC) + source_type = VKD3D_SHADER_SOURCE_DXBC_TPF; + else + source_type = VKD3D_SHADER_SOURCE_D3D_BYTECODE; + + compile_info.type = VKD3D_SHADER_STRUCTURE_TYPE_COMPILE_INFO; + compile_info.next = NULL; + compile_info.source.code = data; + compile_info.source.size = size; + compile_info.source_type = source_type; + compile_info.target_type = VKD3D_SHADER_TARGET_D3D_ASM; + compile_info.options = NULL; + compile_info.option_count = 0; + compile_info.log_level = VKD3D_SHADER_LOG_INFO; + compile_info.source_name = NULL; + + ret = vkd3d_shader_compile(&compile_info, &asm_code, &messages); + + if (ret) + ERR("Failed to disassemble shader, vkd3d result %d.\n", ret); + + if (messages) + { + if (*messages && ERR_ON(d3dcompiler)) + { + const char *ptr = messages; + const char *line; + + ERR("Shader log:\n"); + while ((line = get_line(&ptr))) + { + ERR(" %.*s", (int)(ptr - line), line); + } + ERR("\n"); + } + + vkd3d_shader_free_messages(messages); + } + + if (ret) + return hresult_from_vkd3d_result(ret); + + if (SUCCEEDED(hr = D3DCreateBlob(asm_code.size, disassembly))) + memcpy(ID3D10Blob_GetBufferPointer(*disassembly), asm_code.code, asm_code.size); + + vkd3d_shader_free_shader_code(&asm_code); + + return hr; } HRESULT WINAPI D3DCompileFromFile(const WCHAR *filename, const D3D_SHADER_MACRO *defines, ID3DInclude *include, diff --git a/dlls/d3dcompiler_43/tests/asm.c b/dlls/d3dcompiler_43/tests/asm.c index df686b9378c..6c22eae66c8 100644 --- a/dlls/d3dcompiler_43/tests/asm.c +++ b/dlls/d3dcompiler_43/tests/asm.c @@ -1749,7 +1749,6 @@ static void test_disassemble_shader(void) HRESULT hr; hr = D3DDisassemble(vs_2_0, 0, 0, NULL, &blob); - todo_wine #if D3D_COMPILER_VERSION >= 46 ok(hr == E_INVALIDARG, "Got unexpected hr %#lx.\n", hr); #else @@ -1757,9 +1756,8 @@ static void test_disassemble_shader(void) #endif hr = D3DDisassemble(vs_2_0, sizeof(vs_2_0), 0, NULL, &blob); - todo_wine ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); - if (SUCCEEDED(hr)) - ID3D10Blob_Release(blob); + ok(hr == S_OK, "Got unexpected hr %#lx.\n", hr); + ID3D10Blob_Release(blob); } START_TEST(asm)

1 0

Zhiyi Zhang : winex11.drv: Release window data before calling sync_window_cursor().
by Alexandre Julliard Nov. 30, 2022

Nov. 30, 2022

Module: wine Branch: master Commit: e5c6f915bbd7290641ea43fb41429317bc7e7a80 URL: https://gitlab.winehq.org/wine/wine/-/commit/e5c6f915bbd7290641ea43fb414293… Author: Zhiyi Zhang <zzhang(a)codeweavers.com> Date: Wed Nov 30 12:28:10 2022 +0800 winex11.drv: Release window data before calling sync_window_cursor(). This prevents a deadlock between the window data lock and the ntdll loader lock because sync_window_cursor() eventually may grab the ntdll loader lock. So if another thread already grabbed the ntdll loader lock and wants to enter the same window data critical section, it will deadlock. Fix Youropa (SteamID: 640120) OpenGL launch option launching to a black screen. --- dlls/winex11.drv/mouse.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/dlls/winex11.drv/mouse.c b/dlls/winex11.drv/mouse.c index ebc8be643af..d1ade2eda68 100644 --- a/dlls/winex11.drv/mouse.c +++ b/dlls/winex11.drv/mouse.c @@ -644,6 +644,7 @@ static void map_event_coords( HWND hwnd, Window window, Window event_root, int x static void send_mouse_input( HWND hwnd, Window window, unsigned int state, INPUT *input ) { struct x11drv_win_data *data; + Window win = 0; input->type = INPUT_MOUSE; @@ -665,13 +666,14 @@ static void send_mouse_input( HWND hwnd, Window window, unsigned int state, INPU } if (!(data = get_win_data( hwnd ))) return; + win = data->whole_window; + release_win_data( data ); if (InterlockedExchangePointer( (void **)&cursor_window, hwnd ) != hwnd || input->u.mi.time - last_cursor_change > 100) { - sync_window_cursor( data->whole_window ); + sync_window_cursor( win ); last_cursor_change = input->u.mi.time; } - release_win_data( data ); if (hwnd != NtUserGetDesktopWindow()) {

1 0

Rémi Bernon : joy.cpl: Draw dinput axes state with gdi32.
by Alexandre Julliard Nov. 30, 2022

Nov. 30, 2022

1 0

Rémi Bernon : joy.cpl: Draw dinput button state with gdi32.
by Alexandre Julliard Nov. 30, 2022

Nov. 30, 2022

1 0

Rémi Bernon : joy.cpl: Add a checkbox to test xinput rumble.
by Alexandre Julliard Nov. 30, 2022

Nov. 30, 2022

1 0

Rémi Bernon : joy.cpl: Draw xinput controller state with gdi32.
by Alexandre Julliard Nov. 30, 2022

Nov. 30, 2022

Module: wine Branch: master Commit: 3bc30c372378f27864d35e7681de59ce046ef81f URL: https://gitlab.winehq.org/wine/wine/-/commit/3bc30c372378f27864d35e7681de59… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Sun Nov 27 15:31:11 2022 +0100 joy.cpl: Draw xinput controller state with gdi32. --- dlls/joy.cpl/Makefile.in | 2 +- dlls/joy.cpl/joy_private.h | 1 + dlls/joy.cpl/main.c | 19 ++++ dlls/joy.cpl/xinput.c | 238 ++++++++++++++++++++++++++++++++++++++++++++- 4 files changed, 258 insertions(+), 2 deletions(-)

1 0

Rémi Bernon : joy.cpl: Add a new xinput controller test tab.
by Alexandre Julliard Nov. 30, 2022

Nov. 30, 2022

1 0