January 2024 - Wine-commits

Tim Clem : winemac.drv: Detect active handwriting and panel IMEs.
by Alexandre Julliard 23 Jan '24

23 Jan '24

Module: wine Branch: master Commit: d81c4ce1ba700cca040afcf89c75c683b23e531d URL: https://gitlab.winehq.org/wine/wine/-/commit/d81c4ce1ba700cca040afcf89c75c6… Author: Tim Clem <tclem(a)codeweavers.com> Date: Tue Jan 23 10:33:10 2024 -0800 winemac.drv: Detect active handwriting and panel IMEs. Allows such IMEs that process keyboard input to do so. For instance, fixes backspace and escape for handwriting methods. --- dlls/winemac.drv/cocoa_app.h | 1 - dlls/winemac.drv/cocoa_app.m | 68 +++++++++++++++++++++++++++++++++++--------- 2 files changed, 55 insertions(+), 14 deletions(-) diff --git a/dlls/winemac.drv/cocoa_app.h b/dlls/winemac.drv/cocoa_app.h index c4c8bbd16f2..b2b8187e0b4 100644 --- a/dlls/winemac.drv/cocoa_app.h +++ b/dlls/winemac.drv/cocoa_app.h @@ -92,7 +92,6 @@ enum { CGEventSourceKeyboardType keyboardType; NSEvent* lastFlagsChanged; BOOL inputSourceIsInputMethod; - BOOL inputSourceIsInputMethodValid; uint32_t pressedKeyCodes[128 / 32]; CGFloat primaryScreenHeight; diff --git a/dlls/winemac.drv/cocoa_app.m b/dlls/winemac.drv/cocoa_app.m index 4e26bf1e122..de3a23a5d2d 100644 --- a/dlls/winemac.drv/cocoa_app.m +++ b/dlls/winemac.drv/cocoa_app.m @@ -478,8 +478,6 @@ static NSString* WineLocalizedString(unsigned int stringID) CFRelease(lastKeyboardLayoutInputSource); lastKeyboardLayoutInputSource = inputSourceLayout; - inputSourceIsInputMethodValid = FALSE; - if (inputSourceLayout) { CFDataRef uchr; @@ -2064,24 +2062,68 @@ static NSString* WineLocalizedString(unsigned int stringID) [NSApp activate]; } + static BOOL InputSourceShouldBeIgnored(TISInputSourceRef inputSource) + { + /* Certain system utilities are technically input sources, but we + shouldn't consider them as such for our purposes. + Dictation is its own source too (com.apple.inputmethod.ironwood), but + it should receive keypresses; it cancels input on escape. */ + static CFStringRef ignoredIDs[] = { + /* The "Emoji & Symbols" palette. */ + CFSTR("com.apple.CharacterPaletteIM"), + /* The on-screen keyboard and accessibility panel. */ + CFSTR("com.apple.inputmethod.AssistiveControl"), + /* The popup for accented characters when you hold down a key. */ + CFSTR("com.apple.PressAndHold"), + }; + + CFStringRef sourceID = TISGetInputSourceProperty(inputSource, kTISPropertyInputSourceID); + for (int i = 0; i < sizeof(ignoredIDs) / sizeof(CFStringRef); i++) + { + if (CFEqual(sourceID, ignoredIDs[i])) + return YES; + } + + return NO; + } + - (BOOL) inputSourceIsInputMethod { - if (!inputSourceIsInputMethodValid) + static dispatch_once_t onceToken; + static CFDictionaryRef filterDict; + CFArrayRef enabledSources; + CFIndex i; + BOOL ret = NO; + + /* There may be multiple active ("selected") input sources, but there is + always exactly one selected keyboard input source. For instance, + handwriting methods are active simultaneously with a keyboard source. + As the name implies, TISCopyCurrentKeyboardInputSource only returns + the keyboard source, so it's not sufficient for our needs. We use + TISCreateInputSourceList instead to find all selected sources. */ + dispatch_once(&onceToken, ^{ + filterDict = CFDictionaryCreate(NULL, (const void **)&kTISPropertyInputSourceIsSelected, (const void **)&kCFBooleanTrue, 1, + &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks); + + }); + enabledSources = TISCreateInputSourceList(filterDict, false); + for (i = 0; i < CFArrayGetCount(enabledSources); i++) { - TISInputSourceRef inputSource = TISCopyCurrentKeyboardInputSource(); - if (inputSource) + TISInputSourceRef source = (TISInputSourceRef)CFArrayGetValueAtIndex(enabledSources, i); + CFStringRef type = TISGetInputSourceProperty(source, kTISPropertyInputSourceType); + + /* kTISTypeKeyboardLayout is for physical keyboards. Any type other + than that is an IME. */ + if (!CFEqual(type, kTISTypeKeyboardLayout) && !InputSourceShouldBeIgnored(source)) { - CFStringRef type = TISGetInputSourceProperty(inputSource, kTISPropertyInputSourceType); - inputSourceIsInputMethod = !CFEqual(type, kTISTypeKeyboardLayout); - CFRelease(inputSource); + ret = YES; + break; } - else - inputSourceIsInputMethod = FALSE; - inputSourceIsInputMethodValid = TRUE; } - return inputSourceIsInputMethod; - } + CFRelease(enabledSources); + return ret; + } - (void) releaseMouseCapture {

1 0

Alexandre Julliard : ntdll: Move the process breakpoint to the CPU backends.
by Alexandre Julliard 23 Jan '24

23 Jan '24

Module: wine Branch: master Commit: 1414adfa4644f2cb3b0754aa6a4a54274a9c0310 URL: https://gitlab.winehq.org/wine/wine/-/commit/1414adfa4644f2cb3b0754aa6a4a54… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Jan 23 15:53:58 2024 +0100 ntdll: Move the process breakpoint to the CPU backends. --- dlls/ntdll/loader.c | 30 ++++-------------------------- dlls/ntdll/ntdll_misc.h | 1 + dlls/ntdll/process.c | 13 +------------ dlls/ntdll/signal_arm.c | 16 ++++++++++++++++ dlls/ntdll/signal_arm64.c | 18 ++++++++++++++++++ dlls/ntdll/signal_arm64ec.c | 17 +++++++++++++++++ dlls/ntdll/signal_i386.c | 17 +++++++++++++++++ dlls/ntdll/signal_x86_64.c | 17 +++++++++++++++++ 8 files changed, 91 insertions(+), 38 deletions(-) diff --git a/dlls/ntdll/loader.c b/dlls/ntdll/loader.c index 59624bc70f8..021f7941969 100644 --- a/dlls/ntdll/loader.c +++ b/dlls/ntdll/loader.c @@ -4076,30 +4076,6 @@ PIMAGE_NT_HEADERS WINAPI RtlImageNtHeader(HMODULE hModule) return ret; } -/*********************************************************************** - * process_breakpoint - * - * Trigger a debug breakpoint if the process is being debugged. - */ -static void process_breakpoint(void) -{ - DWORD_PTR port = 0; - - NtQueryInformationProcess( GetCurrentProcess(), ProcessDebugPort, &port, sizeof(port), NULL ); - if (!port) return; - - __TRY - { - DbgBreakPoint(); - } - __EXCEPT_ALL - { - /* do nothing */ - } - __ENDTRY -} - - /*********************************************************************** * load_global_options */ @@ -4270,7 +4246,7 @@ void loader_init( CONTEXT *context, void **entry ) { static int attach_done; NTSTATUS status; - ULONG_PTR cookie; + ULONG_PTR cookie, port = 0; WINE_MODREF *wm; if (process_detaching) NtTerminateThread( GetCurrentThread(), 0 ); @@ -4380,7 +4356,9 @@ void loader_init( CONTEXT *context, void **entry ) release_address_space(); if (wm->ldr.TlsIndex == -1) call_tls_callbacks( wm->ldr.DllBase, DLL_PROCESS_ATTACH ); if (wm->ldr.ActivationContext) RtlDeactivateActivationContext( 0, cookie ); - process_breakpoint(); + + NtQueryInformationProcess( GetCurrentProcess(), ProcessDebugPort, &port, sizeof(port), NULL ); + if (port) process_breakpoint(); } else { diff --git a/dlls/ntdll/ntdll_misc.h b/dlls/ntdll/ntdll_misc.h index da49c14e682..299363d7363 100644 --- a/dlls/ntdll/ntdll_misc.h +++ b/dlls/ntdll/ntdll_misc.h @@ -54,6 +54,7 @@ extern EXCEPTION_DISPOSITION WINAPI user_callback_handler( EXCEPTION_RECORD *rec CONTEXT *context, void *dispatch ); extern void DECLSPEC_NORETURN raise_status( NTSTATUS status, EXCEPTION_RECORD *rec ); extern LONG WINAPI call_unhandled_exception_filter( PEXCEPTION_POINTERS eptr ); +extern void WINAPI process_breakpoint(void); extern void WINAPI LdrInitializeThunk(CONTEXT*,ULONG_PTR,ULONG_PTR,ULONG_PTR); extern NTSTATUS WINAPI KiUserExceptionDispatcher(EXCEPTION_RECORD*,CONTEXT*); diff --git a/dlls/ntdll/process.c b/dlls/ntdll/process.c index 02e31ea7f4b..87090781bbd 100644 --- a/dlls/ntdll/process.c +++ b/dlls/ntdll/process.c @@ -708,18 +708,7 @@ NTSTATUS WINAPI DbgUiConvertStateChangeStructure( DBGUI_WAIT_STATE_CHANGE *state void WINAPI DbgUiRemoteBreakin( void *arg ) { TRACE( "\n" ); - if (NtCurrentTeb()->Peb->BeingDebugged) - { - __TRY - { - DbgBreakPoint(); - } - __EXCEPT_ALL - { - /* do nothing */ - } - __ENDTRY - } + if (NtCurrentTeb()->Peb->BeingDebugged) process_breakpoint(); RtlExitUserThread( STATUS_SUCCESS ); } diff --git a/dlls/ntdll/signal_arm.c b/dlls/ntdll/signal_arm.c index 428742b116b..0ce63932919 100644 --- a/dlls/ntdll/signal_arm.c +++ b/dlls/ntdll/signal_arm.c @@ -1616,6 +1616,22 @@ void WINAPI LdrInitializeThunk( CONTEXT *context, ULONG_PTR unk2, ULONG_PTR unk3 NtContinue( context, TRUE ); } +/*********************************************************************** + * process_breakpoint + */ +void WINAPI process_breakpoint(void) +{ + __TRY + { + DbgBreakPoint(); + } + __EXCEPT_ALL + { + /* do nothing */ + } + __ENDTRY +} + /********************************************************************** * DbgBreakPoint (NTDLL.@) */ diff --git a/dlls/ntdll/signal_arm64.c b/dlls/ntdll/signal_arm64.c index 2d150528610..34cbffbd3cc 100644 --- a/dlls/ntdll/signal_arm64.c +++ b/dlls/ntdll/signal_arm64.c @@ -1585,6 +1585,24 @@ void WINAPI LdrInitializeThunk( CONTEXT *context, ULONG_PTR unk2, ULONG_PTR unk3 NtContinue( context, TRUE ); } + +/*********************************************************************** + * process_breakpoint + */ +void WINAPI process_breakpoint(void) +{ + __TRY + { + DbgBreakPoint(); + } + __EXCEPT_ALL + { + /* do nothing */ + } + __ENDTRY +} + + /********************************************************************** * DbgBreakPoint (NTDLL.@) */ diff --git a/dlls/ntdll/signal_arm64ec.c b/dlls/ntdll/signal_arm64ec.c index a54a5eeaceb..f45381c89c0 100644 --- a/dlls/ntdll/signal_arm64ec.c +++ b/dlls/ntdll/signal_arm64ec.c @@ -2001,6 +2001,23 @@ void WINAPI LdrInitializeThunk( CONTEXT *arm_context, ULONG_PTR unk2, ULONG_PTR } +/*********************************************************************** + * process_breakpoint + */ +void WINAPI process_breakpoint(void) +{ + __TRY + { + DbgBreakPoint(); + } + __EXCEPT_ALL + { + /* do nothing */ + } + __ENDTRY +} + + /********************************************************************** * DbgBreakPoint (NTDLL.@) */ diff --git a/dlls/ntdll/signal_i386.c b/dlls/ntdll/signal_i386.c index 865faa93ea9..95efec5122c 100644 --- a/dlls/ntdll/signal_i386.c +++ b/dlls/ntdll/signal_i386.c @@ -629,6 +629,23 @@ void WINAPI LdrInitializeThunk( CONTEXT *context, ULONG_PTR unk2, ULONG_PTR unk3 } +/*********************************************************************** + * process_breakpoint + */ +void WINAPI process_breakpoint(void) +{ + __TRY + { + DbgBreakPoint(); + } + __EXCEPT_ALL + { + /* do nothing */ + } + __ENDTRY +} + + /********************************************************************** * DbgBreakPoint (NTDLL.@) */ diff --git a/dlls/ntdll/signal_x86_64.c b/dlls/ntdll/signal_x86_64.c index b253c0b4f17..fbab26a7bf4 100644 --- a/dlls/ntdll/signal_x86_64.c +++ b/dlls/ntdll/signal_x86_64.c @@ -1725,6 +1725,23 @@ void WINAPI LdrInitializeThunk( CONTEXT *context, ULONG_PTR unk2, ULONG_PTR unk3 } +/*********************************************************************** + * process_breakpoint + */ +void WINAPI process_breakpoint(void) +{ + __TRY + { + DbgBreakPoint(); + } + __EXCEPT_ALL + { + /* do nothing */ + } + __ENDTRY +} + + /********************************************************************** * DbgBreakPoint (NTDLL.@) */

1 0

Alexandre Julliard : ntdll: Use a .seh handler for the process breakpoint.
by Alexandre Julliard 23 Jan '24

23 Jan '24

Module: wine Branch: master Commit: 6085ab882240fe210d5749eff1d289a44e9245ff URL: https://gitlab.winehq.org/wine/wine/-/commit/6085ab882240fe210d5749eff1d289… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Jan 23 15:55:47 2024 +0100 ntdll: Use a .seh handler for the process breakpoint. --- dlls/ntdll/signal_arm.c | 14 ++++++++++++++ dlls/ntdll/signal_arm64.c | 24 +++++++++++------------- dlls/ntdll/signal_arm64ec.c | 24 +++++++++++------------- dlls/ntdll/signal_x86_64.c | 12 ++++++++++++ 4 files changed, 48 insertions(+), 26 deletions(-) diff --git a/dlls/ntdll/signal_arm.c b/dlls/ntdll/signal_arm.c index 0ce63932919..17a2b8caa08 100644 --- a/dlls/ntdll/signal_arm.c +++ b/dlls/ntdll/signal_arm.c @@ -1619,6 +1619,19 @@ void WINAPI LdrInitializeThunk( CONTEXT *context, ULONG_PTR unk2, ULONG_PTR unk3 /*********************************************************************** * process_breakpoint */ +#ifdef __WINE_PE_BUILD +__ASM_GLOBAL_FUNC( process_breakpoint, + ".seh_endprologue\n\t" + ".seh_handler process_breakpoint_handler, %except\n\t" + "udf #0xfe\n\t" + "bx lr\n" + "process_breakpoint_handler:\n\t" + "ldr r0, [r2, #0x40]\n\t" /* context->Pc */ + "add r0, r0, #2\n\t" + "str r0, [r2, #0x40]\n\t" + "mov r0, #0\n\t" /* ExceptionContinueExecution */ + "bx lr" ) +#else void WINAPI process_breakpoint(void) { __TRY @@ -1631,6 +1644,7 @@ void WINAPI process_breakpoint(void) } __ENDTRY } +#endif /********************************************************************** * DbgBreakPoint (NTDLL.@) diff --git a/dlls/ntdll/signal_arm64.c b/dlls/ntdll/signal_arm64.c index 34cbffbd3cc..2cf7e5fd855 100644 --- a/dlls/ntdll/signal_arm64.c +++ b/dlls/ntdll/signal_arm64.c @@ -1589,19 +1589,17 @@ void WINAPI LdrInitializeThunk( CONTEXT *context, ULONG_PTR unk2, ULONG_PTR unk3 /*********************************************************************** * process_breakpoint */ -void WINAPI process_breakpoint(void) -{ - __TRY - { - DbgBreakPoint(); - } - __EXCEPT_ALL - { - /* do nothing */ - } - __ENDTRY -} - +__ASM_GLOBAL_FUNC( process_breakpoint, + ".seh_endprologue\n\t" + ".seh_handler process_breakpoint_handler, @except\n\t" + "brk #0xf000\n\t" + "ret\n" + "process_breakpoint_handler:\n\t" + "ldr x4, [x2, #0x108]\n\t" /* context->Pc */ + "add x4, x4, #4\n\t" + "str x4, [x2, #0x108]\n\t" + "mov w0, #0\n\t" /* ExceptionContinueExecution */ + "ret" ) /********************************************************************** * DbgBreakPoint (NTDLL.@) diff --git a/dlls/ntdll/signal_arm64ec.c b/dlls/ntdll/signal_arm64ec.c index f45381c89c0..97507d34c4f 100644 --- a/dlls/ntdll/signal_arm64ec.c +++ b/dlls/ntdll/signal_arm64ec.c @@ -2004,19 +2004,17 @@ void WINAPI LdrInitializeThunk( CONTEXT *arm_context, ULONG_PTR unk2, ULONG_PTR /*********************************************************************** * process_breakpoint */ -void WINAPI process_breakpoint(void) -{ - __TRY - { - DbgBreakPoint(); - } - __EXCEPT_ALL - { - /* do nothing */ - } - __ENDTRY -} - +__ASM_GLOBAL_FUNC( "#process_breakpoint", + ".seh_endprologue\n\t" + ".seh_handler process_breakpoint_handler, @except\n\t" + "brk #0xf000\n\t" + "ret\n" + "process_breakpoint_handler:\n\t" + "ldr x4, [x2, #0x108]\n\t" /* context->Pc */ + "add x4, x4, #4\n\t" + "str x4, [x2, #0x108]\n\t" + "mov w0, #0\n\t" /* ExceptionContinueExecution */ + "ret" ) /********************************************************************** * DbgBreakPoint (NTDLL.@) diff --git a/dlls/ntdll/signal_x86_64.c b/dlls/ntdll/signal_x86_64.c index fbab26a7bf4..1344b89a9ca 100644 --- a/dlls/ntdll/signal_x86_64.c +++ b/dlls/ntdll/signal_x86_64.c @@ -1728,6 +1728,17 @@ void WINAPI LdrInitializeThunk( CONTEXT *context, ULONG_PTR unk2, ULONG_PTR unk3 /*********************************************************************** * process_breakpoint */ +#ifdef __WINE_PE_BUILD +__ASM_GLOBAL_FUNC( process_breakpoint, + ".seh_endprologue\n\t" + ".seh_handler process_breakpoint_handler, @except\n\t" + "int $3\n\t" + "ret\n" + "process_breakpoint_handler:\n\t" + "incq 0xf8(%r8)\n\t" /* context->Rip */ + "xorl %eax,%eax\n\t" /* ExceptionContinueExecution */ + "ret" ) +#else void WINAPI process_breakpoint(void) { __TRY @@ -1740,6 +1751,7 @@ void WINAPI process_breakpoint(void) } __ENDTRY } +#endif /**********************************************************************

1 0

Alexandre Julliard : ntdll/tests: Update todos in context tests for new wow64 mode.
by Alexandre Julliard 23 Jan '24

23 Jan '24

Module: wine Branch: master Commit: 5113e551396049d574bd3d246887dd4229188b88 URL: https://gitlab.winehq.org/wine/wine/-/commit/5113e551396049d574bd3d246887dd… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue Jan 23 22:44:33 2024 +0100 ntdll/tests: Update todos in context tests for new wow64 mode. --- dlls/ntdll/tests/exception.c | 32 +++++++++++++++----------------- 1 file changed, 15 insertions(+), 17 deletions(-) diff --git a/dlls/ntdll/tests/exception.c b/dlls/ntdll/tests/exception.c index 5917196a0cf..a183980895a 100644 --- a/dlls/ntdll/tests/exception.c +++ b/dlls/ntdll/tests/exception.c @@ -1270,7 +1270,7 @@ static void test_debugger(DWORD cont_status, BOOL with_WaitForDebugEventEx) __asm__( "movw %%ss,%0" : "=r" (ss) ); ok( ctx.SegSs == ss, "wrong ss %04lx / %04x\n", ctx.SegSs, ss ); ok( ctx.SegFs != ctx.SegSs, "wrong fs %04lx / %04lx\n", ctx.SegFs, ctx.SegSs ); - if (is_wow64) todo_wine + if (is_wow64) todo_wine_if( !ctx.SegDs ) /* old wow64 */ { ok( ctx.SegDs == ctx.SegSs, "wrong ds %04lx / %04lx\n", ctx.SegDs, ctx.SegSs ); ok( ctx.SegEs == ctx.SegSs, "wrong es %04lx / %04lx\n", ctx.SegEs, ctx.SegSs ); @@ -4547,10 +4547,6 @@ static void test_wow64_context(void) ret = pNtGetContextThread( pi.hThread, &context ); ok(ret == STATUS_SUCCESS, "got %#lx\n", ret); ok( context.ContextFlags == CONTEXT_ALL, "got context flags %#lx\n", context.ContextFlags ); - todo_wine - ok( !context.Rax, "rax is not zero %Ix\n", context.Rax ); - todo_wine - ok( !context.Rbx, "rbx is not zero %Ix\n", context.Rbx ); ok( !context.Rsi, "rsi is not zero %Ix\n", context.Rsi ); ok( !context.Rdi, "rdi is not zero %Ix\n", context.Rdi ); ok( !context.Rbp, "rbp is not zero %Ix\n", context.Rbp ); @@ -4562,15 +4558,8 @@ static void test_wow64_context(void) ok( !context.R13, "r13 is not zero %Ix\n", context.R13 ); ok( !context.R14, "r14 is not zero %Ix\n", context.R14 ); ok( !context.R15, "r15 is not zero %Ix\n", context.R15 ); - todo_wine - ok( ((ULONG_PTR)context.Rsp & ~0xfff) == ((ULONG_PTR)teb.Tib.StackBase & ~0xfff), - "rsp is not at top of stack %p / %p\n", (void *)context.Rsp, teb.Tib.StackBase ); - todo_wine - ok( context.EFlags == 0x200, "wrong flags %08lx\n", context.EFlags ); ok( context.MxCsr == 0x1f80, "wrong mxcsr %08lx\n", context.MxCsr ); ok( context.FltSave.ControlWord == 0x27f, "wrong control %08x\n", context.FltSave.ControlWord ); - todo_wine - ok( context.SegCs != ctx.SegCs, "wrong cs %04x\n", context.SegCs ); ok( context.SegDs == ctx.SegDs, "wrong ds %04x / %04lx\n", context.SegDs, ctx.SegDs ); ok( context.SegEs == ctx.SegEs, "wrong es %04x / %04lx\n", context.SegEs, ctx.SegEs ); ok( context.SegFs == ctx.SegFs, "wrong fs %04x / %04lx\n", context.SegFs, ctx.SegFs ); @@ -4585,6 +4574,12 @@ static void test_wow64_context(void) goto done; } + ok( !context.Rax, "rax is not zero %Ix\n", context.Rax ); + ok( !context.Rbx, "rbx is not zero %Ix\n", context.Rbx ); + ok( ((ULONG_PTR)context.Rsp & ~0xfff) == ((ULONG_PTR)teb.Tib.StackBase & ~0xfff), + "rsp is not at top of stack %p / %p\n", (void *)context.Rsp, teb.Tib.StackBase ); + ok( context.EFlags == 0x200, "wrong flags %08lx\n", context.EFlags ); + for (i = 0, got32 = got64 = FALSE; i < 10000 && !(got32 && got64); i++) { ResumeThread( pi.hThread ); @@ -4661,21 +4656,23 @@ static void test_wow64_context(void) memset( &ctx, 0x55, sizeof(ctx) ); ctx.ContextFlags = WOW64_CONTEXT_ALL; pRtlWow64GetThreadContext( pi.hThread, &ctx ); - ok( ctx.Ecx == 0x87654321, "cs64: ecx set to %08lx\n", ctx.Ecx ); + todo_wine + ok( ctx.Ecx == 0x87654321, "cs32: ecx set to %08lx\n", ctx.Ecx ); ReadProcessMemory( pi.hProcess, teb.TlsSlots[WOW64_TLS_CPURESERVED], cpu, cpu_size, &res ); - ok( ctx_ptr->Ecx == ecx, "cs64: ecx set to %08lx\n", ctx_ptr->Ecx ); + ok( ctx_ptr->Ecx == ecx, "cs32: ecx set to %08lx\n", ctx_ptr->Ecx ); ctx.Ecx = 0x33334444; pRtlWow64SetThreadContext( pi.hThread, &ctx ); memset( &ctx, 0x55, sizeof(ctx) ); ctx.ContextFlags = WOW64_CONTEXT_ALL; pRtlWow64GetThreadContext( pi.hThread, &ctx ); - ok( ctx.Ecx == 0x33334444, "cs64: ecx set to %08lx\n", ctx.Ecx ); + ok( ctx.Ecx == 0x33334444, "cs32: ecx set to %08lx\n", ctx.Ecx ); ReadProcessMemory( pi.hProcess, teb.TlsSlots[WOW64_TLS_CPURESERVED], cpu, cpu_size, &res ); - ok( ctx_ptr->Ecx == ecx, "cs64: ecx set to %08lx\n", ctx_ptr->Ecx ); + ok( ctx_ptr->Ecx == ecx, "cs32: ecx set to %08lx\n", ctx_ptr->Ecx ); memset( &context, 0x55, sizeof(context) ); context.ContextFlags = CONTEXT_ALL; pNtGetContextThread( pi.hThread, &context ); - ok( context.Rcx == 0x33334444, "cs64: rcx set to %p\n", (void *)context.Rcx ); + todo_wine + ok( context.Rcx == 0x33334444, "cs32: rcx set to %p\n", (void *)context.Rcx ); /* restore everything */ context.Rcx = rcx; pNtSetContextThread( pi.hThread, &context ); @@ -4738,6 +4735,7 @@ static void test_wow64_context(void) pRtlWow64GetThreadContext( pi.hThread, &ctx ); ok( ctx.Ecx == 0x22223333, "cs64: ecx set to %08lx\n", ctx.Ecx ); ReadProcessMemory( pi.hProcess, teb.TlsSlots[WOW64_TLS_CPURESERVED], cpu, cpu_size, &res ); + todo_wine ok( ctx_ptr->Ecx == 0x22223333, "cs64: ecx set to %08lx\n", ctx_ptr->Ecx ); memset( &context, 0x55, sizeof(context) ); context.ContextFlags = CONTEXT_ALL;

1 0

Hans Leidekker : bcrypt: Fix private data size in wow64 thunks.
by Alexandre Julliard 23 Jan '24

23 Jan '24

Module: wine Branch: master Commit: 7231d4586d3ceccdf46f13bdd696d2b3ce0dc675 URL: https://gitlab.winehq.org/wine/wine/-/commit/7231d4586d3ceccdf46f13bdd696d2… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Mon Jan 22 13:49:37 2024 +0100 bcrypt: Fix private data size in wow64 thunks. --- dlls/bcrypt/bcrypt_internal.h | 3 ++- dlls/bcrypt/gnutls.c | 14 +++++--------- 2 files changed, 7 insertions(+), 10 deletions(-) diff --git a/dlls/bcrypt/bcrypt_internal.h b/dlls/bcrypt/bcrypt_internal.h index f6556eb2aaa..f0157e86a73 100644 --- a/dlls/bcrypt/bcrypt_internal.h +++ b/dlls/bcrypt/bcrypt_internal.h @@ -184,11 +184,12 @@ struct key_asymmetric DSSSEED dss_seed; }; +#define PRIVATE_DATA_SIZE 3 struct key { struct object hdr; enum alg_id alg_id; - UINT64 private[3]; /* private data for backend */ + UINT64 private[PRIVATE_DATA_SIZE]; /* private data for backend */ union { struct key_symmetric s; diff --git a/dlls/bcrypt/gnutls.c b/dlls/bcrypt/gnutls.c index 512731dfcd3..ec1f0fb09ce 100644 --- a/dlls/bcrypt/gnutls.c +++ b/dlls/bcrypt/gnutls.c @@ -2741,7 +2741,7 @@ struct key32 { struct object hdr; enum alg_id alg_id; - UINT64 private[2]; /* private data for backend */ + UINT64 private[PRIVATE_DATA_SIZE]; /* private data for backend */ union { struct key_symmetric32 s; @@ -2791,8 +2791,7 @@ static struct key *get_symmetric_key( struct key32 *key32, struct key *key ) { key->hdr = key32->hdr; key->alg_id = key32->alg_id; - key->private[0] = key32->private[0]; - key->private[1] = key32->private[1]; + memcpy( key->private, key32->private, sizeof(key->private) ); key->u.s.mode = key32->u.s.mode; key->u.s.block_size = key32->u.s.block_size; key->u.s.vector = ULongToPtr(key32->u.s.vector); @@ -2806,8 +2805,7 @@ static struct key *get_asymmetric_key( struct key32 *key32, struct key *key ) { key->hdr = key32->hdr; key->alg_id = key32->alg_id; - key->private[0] = key32->private[0]; - key->private[1] = key32->private[1]; + memcpy( key->private, key32->private, sizeof(key->private) ); key->u.a.bitlen = key32->u.a.bitlen; key->u.a.flags = key32->u.a.flags; key->u.a.dss_seed = key32->u.a.dss_seed; @@ -2816,14 +2814,12 @@ static struct key *get_asymmetric_key( struct key32 *key32, struct key *key ) static void put_symmetric_key32( struct key *key, struct key32 *key32 ) { - key32->private[0] = key->private[0]; - key32->private[1] = key->private[1]; + memcpy( key32->private, key->private, sizeof(key32->private) ); } static void put_asymmetric_key32( struct key *key, struct key32 *key32 ) { - key32->private[0] = key->private[0]; - key32->private[1] = key->private[1]; + memcpy( key32->private, key->private, sizeof(key32->private) ); key32->u.a.flags = key->u.a.flags; key32->u.a.dss_seed = key->u.a.dss_seed; }

1 0

Hans Leidekker : dssenh: Finalize the hash if necessary in CPVerifySignature().
by Alexandre Julliard 23 Jan '24

23 Jan '24

Module: wine Branch: master Commit: 1d0551ca49fdff68364f7cac42415cea0f586b16 URL: https://gitlab.winehq.org/wine/wine/-/commit/1d0551ca49fdff68364f7cac42415c… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Tue Dec 12 13:14:08 2023 +0100 dssenh: Finalize the hash if necessary in CPVerifySignature(). --- dlls/dssenh/main.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/dlls/dssenh/main.c b/dlls/dssenh/main.c index 6772d062fc5..365b365f53f 100644 --- a/dlls/dssenh/main.c +++ b/dlls/dssenh/main.c @@ -1012,5 +1012,11 @@ BOOL WINAPI CPVerifySignature( HCRYPTPROV hprov, HCRYPTHASH hhash, const BYTE *s return FALSE; } + if (!hash->finished) + { + if (BCryptFinishHash( hash->handle, hash->value, hash->len, 0 )) return FALSE; + hash->finished = TRUE; + } + return !BCryptVerifySignature( key->handle, NULL, hash->value, hash->len, (UCHAR *)sig, siglen, 0 ); }

1 0

Rémi Bernon : dinput/tests: Introduce a new helper to create a foreground window.
by Alexandre Julliard 23 Jan '24

23 Jan '24

Module: wine Branch: master Commit: e2dffb8eb0b68cbd2115d67f970fe38f6fb698eb URL: https://gitlab.winehq.org/wine/wine/-/commit/e2dffb8eb0b68cbd2115d67f970fe3… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Sat Jan 20 19:05:18 2024 +0100 dinput/tests: Introduce a new helper to create a foreground window. --- dlls/dinput/tests/device8.c | 65 +++++++++++++++++++++++++------------- dlls/dinput/tests/dinput_test.h | 3 ++ dlls/dinput/tests/force_feedback.c | 6 ++-- dlls/dinput/tests/joystick8.c | 18 ++--------- 4 files changed, 51 insertions(+), 41 deletions(-)

1 0

Rémi Bernon : dinput/tests: Enforce ordering of concurrent read IRPs.
by Alexandre Julliard 23 Jan '24

23 Jan '24

Module: wine Branch: master Commit: 3e79b8d6ebdb3372749aed67e5ee19eb5fd40586 URL: https://gitlab.winehq.org/wine/wine/-/commit/3e79b8d6ebdb3372749aed67e5ee19… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Sat Jan 20 22:01:11 2024 +0100 dinput/tests: Enforce ordering of concurrent read IRPs. --- dlls/dinput/tests/driver_bus.c | 30 +++++++++++++++++++----------- 1 file changed, 19 insertions(+), 11 deletions(-) diff --git a/dlls/dinput/tests/driver_bus.c b/dlls/dinput/tests/driver_bus.c index c0d181c58a5..e0943c35294 100644 --- a/dlls/dinput/tests/driver_bus.c +++ b/dlls/dinput/tests/driver_bus.c @@ -348,13 +348,13 @@ static void irp_queue_push( struct irp_queue *queue, IRP *irp ) KeReleaseSpinLock( &queue->lock, irql ); } -static void irp_queue_clear( struct irp_queue *queue ) +static void irp_queue_complete( struct irp_queue *queue, BOOL delete ) { IRP *irp; while ((irp = irp_queue_pop( queue ))) { - irp->IoStatus.Status = STATUS_DELETE_PENDING; + if (delete) irp->IoStatus.Status = STATUS_DELETE_PENDING; IoCompleteRequest( irp, IO_NO_INCREMENT ); } } @@ -373,6 +373,7 @@ struct input_queue struct hid_expect *end; struct hid_expect *buffer; struct irp_queue pending; + struct irp_queue completed; }; static void input_queue_init( struct input_queue *queue, BOOL is_polled ) @@ -384,6 +385,7 @@ static void input_queue_init( struct input_queue *queue, BOOL is_polled ) queue->pos = queue->buffer; queue->end = queue->buffer; irp_queue_init( &queue->pending ); + irp_queue_init( &queue->completed ); } static void input_queue_cleanup( struct input_queue *queue ) @@ -416,7 +418,11 @@ static NTSTATUS input_queue_read( struct input_queue *queue, IRP *irp ) KIRQL irql; KeAcquireSpinLock( &queue->lock, &irql ); - if (input_queue_read_locked( queue, irp )) status = STATUS_SUCCESS; + if (input_queue_read_locked( queue, irp )) + { + irp_queue_push( &queue->completed, irp ); + status = STATUS_SUCCESS; + } else { IoMarkIrpPending( irp ); @@ -430,13 +436,10 @@ static NTSTATUS input_queue_read( struct input_queue *queue, IRP *irp ) static void input_queue_reset( struct input_queue *queue, void *in_buf, ULONG in_size ) { - struct irp_queue completed; ULONG remaining; KIRQL irql; IRP *irp; - irp_queue_init( &completed ); - KeAcquireSpinLock( &queue->lock, &irql ); remaining = queue->end - queue->pos; queue->pos = queue->buffer; @@ -447,13 +450,13 @@ static void input_queue_reset( struct input_queue *queue, void *in_buf, ULONG in while (!queue->is_polled && queue->pos < queue->end && (irp = irp_queue_pop( &queue->pending ))) { input_queue_read_locked( queue, irp ); - irp_queue_push( &completed, irp ); + irp_queue_push( &queue->completed, irp ); } KeReleaseSpinLock( &queue->lock, irql ); if (!queue->is_polled) ok( !remaining, "unread input\n" ); - while ((irp = irp_queue_pop( &completed ))) IoCompleteRequest( irp, IO_NO_INCREMENT ); + irp_queue_complete( &queue->completed, FALSE ); } struct device @@ -721,7 +724,8 @@ static NTSTATUS pdo_pnp( DEVICE_OBJECT *device, IRP *irp ) state = (code == IRP_MN_START_DEVICE || code == IRP_MN_CANCEL_REMOVE_DEVICE) ? 0 : PNP_DEVICE_REMOVED; KeAcquireSpinLock( &impl->base.lock, &irql ); impl->base.state = state; - irp_queue_clear( &impl->input_queue.pending ); + irp_queue_complete( &impl->input_queue.pending, TRUE ); + irp_queue_complete( &impl->input_queue.completed, TRUE ); KeReleaseSpinLock( &impl->base.lock, irql ); if (code != IRP_MN_REMOVE_DEVICE) status = STATUS_SUCCESS; else @@ -1087,7 +1091,10 @@ static NTSTATUS pdo_internal_ioctl( DEVICE_OBJECT *device, IRP *irp ) ok( !in_size, "got input size %lu\n", in_size ); ok( out_size == expected_size, "got output size %lu\n", out_size ); status = input_queue_read( &impl->input_queue, irp ); - break; + irp_queue_complete( &impl->input_queue.completed, FALSE ); + + winetest_pop_context(); + return status; } case IOCTL_HID_WRITE_REPORT: @@ -1277,7 +1284,8 @@ static NTSTATUS pdo_handle_ioctl( struct phys_device *impl, IRP *irp, ULONG code case IOCTL_WINETEST_REMOVE_DEVICE: KeAcquireSpinLock( &impl->base.lock, &irql ); impl->base.state = PNP_DEVICE_REMOVED; - irp_queue_clear( &impl->input_queue.pending ); + irp_queue_complete( &impl->input_queue.pending, TRUE ); + irp_queue_complete( &impl->input_queue.completed, TRUE ); KeReleaseSpinLock( &impl->base.lock, irql ); impl->pending_remove = irp; IoMarkIrpPending( irp );

1 0

Rémi Bernon : dinput/tests: Add a test with a virtual HID mouse.
by Alexandre Julliard 23 Jan '24

23 Jan '24

Module: wine Branch: master Commit: e2f2f680d464a7d7d48b3c3be1d32e3ea1b2d541 URL: https://gitlab.winehq.org/wine/wine/-/commit/e2f2f680d464a7d7d48b3c3be1d32e… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Thu Jun 2 13:30:25 2022 +0200 dinput/tests: Add a test with a virtual HID mouse. --- dlls/dinput/tests/device8.c | 149 ++++++++++++++++++++++++++++++++++++++++ dlls/dinput/tests/dinput_test.h | 4 +- dlls/dinput/tests/hid.c | 6 +- 3 files changed, 155 insertions(+), 4 deletions(-) diff --git a/dlls/dinput/tests/device8.c b/dlls/dinput/tests/device8.c index 7423d6c6ae7..e3cd75f7c08 100644 --- a/dlls/dinput/tests/device8.c +++ b/dlls/dinput/tests/device8.c @@ -1618,6 +1618,149 @@ cleanup: localized = old_localized; } +static UINT mouse_move_count; + +static LRESULT CALLBACK mouse_wndproc( HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam ) +{ + if (msg == WM_MOUSEMOVE) + { + ok( wparam == 0, "got wparam %#Ix\n", wparam ); + ok( LOWORD(lparam) - 107 <= 2, "got LOWORD(lparam) %u\n", LOWORD(lparam) ); + ok( HIWORD(lparam) - 107 <= 2, "got HIWORD(lparam) %u\n", HIWORD(lparam) ); + mouse_move_count++; + } + return DefWindowProcA( hwnd, msg, wparam, lparam ); +} + +static void test_hid_mouse(void) +{ +#include "psh_hid_macros.h" + const unsigned char report_desc[] = + { + USAGE_PAGE(1, HID_USAGE_PAGE_GENERIC), + USAGE(1, HID_USAGE_GENERIC_MOUSE), + COLLECTION(1, Application), + USAGE(1, HID_USAGE_GENERIC_POINTER), + COLLECTION(1, Physical), + REPORT_ID(1, 1), + + USAGE_PAGE(1, HID_USAGE_PAGE_BUTTON), + USAGE_MINIMUM(1, 1), + USAGE_MAXIMUM(1, 2), + LOGICAL_MINIMUM(1, 0), + LOGICAL_MAXIMUM(1, 1), + REPORT_SIZE(1, 1), + REPORT_COUNT(1, 8), + INPUT(1, Data|Var|Abs), + + USAGE_PAGE(1, HID_USAGE_PAGE_GENERIC), + USAGE(1, HID_USAGE_GENERIC_X), + USAGE(1, HID_USAGE_GENERIC_Y), + REPORT_SIZE(1, 16), + REPORT_COUNT(1, 2), + LOGICAL_MINIMUM(2, -10), + LOGICAL_MAXIMUM(2, +10), + INPUT(1, Data|Var|Rel), + END_COLLECTION, + END_COLLECTION, + }; + C_ASSERT(sizeof(report_desc) < MAX_HID_DESCRIPTOR_LEN); +#include "pop_hid_macros.h" + + const HID_DEVICE_ATTRIBUTES attributes = + { + .Size = sizeof(HID_DEVICE_ATTRIBUTES), + .VendorID = LOWORD(EXPECT_VIDPID), + .ProductID = 0x0002, + .VersionNumber = 0x0100, + }; + struct hid_device_desc desc = + { + .caps = { .InputReportByteLength = 6 }, + .attributes = attributes, + .use_report_id = 1, + }; + struct hid_expect single_move[] = + { + { + .code = IOCTL_HID_READ_REPORT, + .report_buf = {1,0x00,0x08,0x00,0x08,0x00}, + }, + }; + + WCHAR device_path[MAX_PATH]; + HANDLE file; + POINT pos; + DWORD res; + HWND hwnd; + BOOL ret; + + winetest_push_context( "mouse "); + + desc.report_descriptor_len = sizeof(report_desc); + memcpy( desc.report_descriptor_buf, report_desc, sizeof(report_desc) ); + fill_context( desc.context, ARRAY_SIZE(desc.context) ); + + if (!hid_device_start_( &desc, 1, 5000 /* needs a long timeout on Win7 */ )) goto done; + + swprintf( device_path, MAX_PATH, L"\\\\?\\hid#vid_%04x&pid_%04x", desc.attributes.VendorID, + desc.attributes.ProductID ); + ret = find_hid_device_path( device_path ); + ok( ret, "Failed to find HID device matching %s\n", debugstr_w( device_path ) ); + + + /* windows doesn't let us open HID mouse devices */ + + file = CreateFileW( device_path, FILE_READ_ACCESS | FILE_WRITE_ACCESS, + FILE_SHARE_READ | FILE_SHARE_WRITE, NULL, OPEN_EXISTING, 0, NULL ); + todo_wine + ok( file == INVALID_HANDLE_VALUE, "CreateFileW succeeded\n" ); + todo_wine + ok( GetLastError() == ERROR_ACCESS_DENIED, "got error %lu\n", GetLastError() ); + CloseHandle( file ); + + file = CreateFileW( L"\\\\?\\root#winetest#0#{deadbeef-29ef-4538-a5fd-b69573a362c0}", 0, 0, + NULL, OPEN_EXISTING, 0, NULL ); + ok( file != INVALID_HANDLE_VALUE, "CreateFile failed, error %lu\n", GetLastError() ); + + + /* check basic mouse input injection to window message */ + + SetCursorPos( 100, 100 ); + hwnd = create_foreground_window( TRUE ); + SetWindowLongPtrW( hwnd, GWLP_WNDPROC, (LONG_PTR)mouse_wndproc ); + + GetCursorPos( &pos ); + ok( pos.x == 100, "got x %lu\n", pos.x ); + ok( pos.y == 100, "got y %lu\n", pos.y ); + + mouse_move_count = 0; + bus_send_hid_input( file, &desc, single_move, sizeof(single_move) ); + + res = MsgWaitForMultipleObjects( 0, NULL, FALSE, 500, QS_MOUSEMOVE ); + todo_wine + ok( !res, "MsgWaitForMultipleObjects returned %#lx\n", res ); + msg_wait_for_events( 0, NULL, 5 ); /* process pending messages */ + todo_wine + ok( mouse_move_count == 1, "got mouse_move_count %u\n", mouse_move_count ); + + GetCursorPos( &pos ); + todo_wine + ok( (ULONG)pos.x - 107 <= 2, "got x %lu\n", pos.x ); + todo_wine + ok( (ULONG)pos.y - 107 <= 2, "got y %lu\n", pos.y ); + + DestroyWindow( hwnd ); + + + CloseHandle( file ); + +done: + hid_device_stop( &desc, 1 ); + + winetest_pop_context(); +} + static void test_dik_codes( IDirectInputDevice8W *device, HANDLE event, HWND hwnd, DWORD version ) { static const struct key2dik @@ -2371,5 +2514,11 @@ START_TEST(device8) test_keyboard_events(); test_appdata_property(); + if (!bus_device_start()) goto done; + + test_hid_mouse(); + +done: + bus_device_stop(); dinput_test_exit(); } diff --git a/dlls/dinput/tests/dinput_test.h b/dlls/dinput/tests/dinput_test.h index b53978bbe97..e13ba2421d2 100644 --- a/dlls/dinput/tests/dinput_test.h +++ b/dlls/dinput/tests/dinput_test.h @@ -51,11 +51,13 @@ extern const WCHAR expect_path_end[]; extern HINSTANCE instance; extern BOOL localized; /* object names get translated */ -BOOL hid_device_start( struct hid_device_desc *desc, UINT count ); +#define hid_device_start( a, b ) hid_device_start_( a, b, 1000 ) +BOOL hid_device_start_( struct hid_device_desc *desc, UINT count, DWORD timeout ); void hid_device_stop( struct hid_device_desc *desc, UINT count ); BOOL bus_device_start(void); void bus_device_stop(void); +BOOL find_hid_device_path( WCHAR *device_path ); void cleanup_registry_keys(void); #define dinput_test_init() dinput_test_init_( __FILE__, __LINE__ ) diff --git a/dlls/dinput/tests/hid.c b/dlls/dinput/tests/hid.c index 0a28fda6f99..e5f6ea4640d 100644 --- a/dlls/dinput/tests/hid.c +++ b/dlls/dinput/tests/hid.c @@ -553,7 +553,7 @@ void bus_device_stop(void) ok( ret || GetLastError() == ERROR_FILE_NOT_FOUND, "Failed to delete file, error %lu\n", GetLastError() ); } -static BOOL find_hid_device_path( WCHAR *device_path ) +BOOL find_hid_device_path( WCHAR *device_path ) { char buffer[FIELD_OFFSET( SP_DEVICE_INTERFACE_DETAIL_DATA_W, DevicePath[MAX_PATH] )] = {0}; SP_DEVICE_INTERFACE_DATA iface = {sizeof(SP_DEVICE_INTERFACE_DATA)}; @@ -735,7 +735,7 @@ void hid_device_stop( struct hid_device_desc *desc, UINT count ) } } -BOOL hid_device_start( struct hid_device_desc *desc, UINT count ) +BOOL hid_device_start_( struct hid_device_desc *desc, UINT count, DWORD timeout ) { HANDLE control; DWORD ret, i; @@ -754,7 +754,7 @@ BOOL hid_device_start( struct hid_device_desc *desc, UINT count ) for (i = 0; i < count; ++i) { - ret = WaitForSingleObject( device_added, 1000 ); + ret = WaitForSingleObject( device_added, timeout ); todo_wine_if(i > 0) ok( !ret, "WaitForSingleObject returned %#lx\n", ret ); }

1 0

Rémi Bernon : dinput/tests: Add a test with a virtual HID keyboard.
by Alexandre Julliard 23 Jan '24

23 Jan '24

Module: wine Branch: master Commit: abee0d45021f1be3626c1c2d91fb48284b0ea272 URL: https://gitlab.winehq.org/wine/wine/-/commit/abee0d45021f1be3626c1c2d91fb48… Author: Rémi Bernon <rbernon(a)codeweavers.com> Date: Fri May 19 17:37:41 2023 +0200 dinput/tests: Add a test with a virtual HID keyboard. --- dlls/dinput/tests/device8.c | 148 ++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 148 insertions(+) diff --git a/dlls/dinput/tests/device8.c b/dlls/dinput/tests/device8.c index e3cd75f7c08..594a34222ce 100644 --- a/dlls/dinput/tests/device8.c +++ b/dlls/dinput/tests/device8.c @@ -2488,6 +2488,153 @@ static void test_sys_keyboard_action_format(void) ok( ref == 0, "Release returned %ld\n", ref ); } +static UINT key_down_count; +static UINT key_up_count; + +static LRESULT CALLBACK keyboard_wndproc( HWND hwnd, UINT msg, WPARAM wparam, LPARAM lparam ) +{ + BOOL us_kbd = (GetKeyboardLayout(0) == (HKL)(ULONG_PTR)0x04090409); + + if (msg == WM_KEYDOWN) + { + key_down_count++; + if (us_kbd) ok( wparam == 'A', "got wparam %#Ix\n", wparam ); + ok( lparam == 0x1e0001, "got lparam %#Ix\n", lparam ); + } + if (msg == WM_KEYUP) + { + key_up_count++; + if (us_kbd) ok( wparam == 'A', "got wparam %#Ix\n", wparam ); + ok( lparam == 0xc01e0001, "got lparam %#Ix\n", lparam ); + } + return DefWindowProcA( hwnd, msg, wparam, lparam ); +} + +static void test_hid_keyboard(void) +{ +#include "psh_hid_macros.h" + const unsigned char report_desc[] = + { + USAGE_PAGE(1, HID_USAGE_PAGE_GENERIC), + USAGE(1, HID_USAGE_GENERIC_KEYBOARD), + COLLECTION(1, Application), + USAGE(1, HID_USAGE_GENERIC_KEYBOARD), + COLLECTION(1, Physical), + REPORT_ID(1, 1), + + USAGE_PAGE(1, HID_USAGE_PAGE_KEYBOARD), + USAGE_MINIMUM(1, HID_USAGE_KEYBOARD_aA), + USAGE_MAXIMUM(1, HID_USAGE_KEYBOARD_zZ), + LOGICAL_MINIMUM(1, 0), + LOGICAL_MAXIMUM(1, 1), + REPORT_SIZE(1, 1), + REPORT_COUNT(1, 32), + INPUT(1, Data|Var|Abs), + END_COLLECTION, + END_COLLECTION, + }; + C_ASSERT(sizeof(report_desc) < MAX_HID_DESCRIPTOR_LEN); +#include "pop_hid_macros.h" + + const HID_DEVICE_ATTRIBUTES attributes = + { + .Size = sizeof(HID_DEVICE_ATTRIBUTES), + .VendorID = LOWORD(EXPECT_VIDPID), + .ProductID = 0x0003, + .VersionNumber = 0x0100, + }; + struct hid_device_desc desc = + { + .caps = { .InputReportByteLength = 5 }, + .attributes = attributes, + .use_report_id = 1, + }; + struct hid_expect key_press_release[] = + { + { + .code = IOCTL_HID_READ_REPORT, + .report_buf = {1,0x01,0x00,0x00,0x00}, + }, + { + .code = IOCTL_HID_READ_REPORT, + .report_buf = {1,0x00,0x00,0x00,0x00}, + }, + }; + + WCHAR device_path[MAX_PATH]; + HANDLE file; + DWORD res; + HWND hwnd; + BOOL ret; + + winetest_push_context( "keyboard" ); + + desc.report_descriptor_len = sizeof(report_desc); + memcpy( desc.report_descriptor_buf, report_desc, sizeof(report_desc) ); + fill_context( desc.context, ARRAY_SIZE(desc.context) ); + + if (!hid_device_start_( &desc, 1, 5000 /* needs a long timeout on Win7 */ )) goto done; + + swprintf( device_path, MAX_PATH, L"\\\\?\\hid#vid_%04x&pid_%04x", desc.attributes.VendorID, + desc.attributes.ProductID ); + ret = find_hid_device_path( device_path ); + ok( ret, "Failed to find HID device matching %s\n", debugstr_w( device_path ) ); + + + /* windows doesn't let us open HID keyboard devices */ + + file = CreateFileW( device_path, FILE_READ_ACCESS | FILE_WRITE_ACCESS, + FILE_SHARE_READ | FILE_SHARE_WRITE, NULL, OPEN_EXISTING, 0, NULL ); + todo_wine + ok( file == INVALID_HANDLE_VALUE, "CreateFileW succeeded\n" ); + todo_wine + ok( GetLastError() == ERROR_ACCESS_DENIED, "got error %lu\n", GetLastError() ); + CloseHandle( file ); + + file = CreateFileW( L"\\\\?\\root#winetest#0#{deadbeef-29ef-4538-a5fd-b69573a362c0}", 0, 0, + NULL, OPEN_EXISTING, 0, NULL ); + ok( file != INVALID_HANDLE_VALUE, "CreateFile failed, error %lu\n", GetLastError() ); + + + /* check basic keyboard input injection to window message */ + + hwnd = create_foreground_window( FALSE ); + SetWindowLongPtrW( hwnd, GWLP_WNDPROC, (LONG_PTR)keyboard_wndproc ); + + key_down_count = 0; + key_up_count = 0; + bus_send_hid_input( file, &desc, &key_press_release[0], sizeof(key_press_release[0]) ); + + res = MsgWaitForMultipleObjects( 0, NULL, FALSE, 500, QS_KEY ); + todo_wine + ok( !res, "MsgWaitForMultipleObjects returned %#lx\n", res ); + msg_wait_for_events( 0, NULL, 5 ); /* process pending messages */ + todo_wine + ok( key_down_count == 1, "got key_down_count %u\n", key_down_count ); + ok( key_up_count == 0, "got key_up_count %u\n", key_up_count ); + + bus_send_hid_input( file, &desc, &key_press_release[1], sizeof(key_press_release[1]) ); + + res = MsgWaitForMultipleObjects( 0, NULL, FALSE, 500, QS_KEY ); + todo_wine + ok( !res, "MsgWaitForMultipleObjects returned %#lx\n", res ); + msg_wait_for_events( 0, NULL, 5 ); /* process pending messages */ + todo_wine + ok( key_down_count == 1, "got key_down_count %u\n", key_down_count ); + todo_wine + ok( key_up_count == 1, "got key_up_count %u\n", key_up_count ); + + DestroyWindow( hwnd ); + + + CloseHandle( file ); + +done: + hid_device_stop( &desc, 1 ); + + winetest_pop_context(); +} + START_TEST(device8) { dinput_test_init(); @@ -2517,6 +2664,7 @@ START_TEST(device8) if (!bus_device_start()) goto done; test_hid_mouse(); + test_hid_keyboard(); done: bus_device_stop();

1 0