Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2024-06-26 07:51:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

[WIN32K:NTUSER] Do not pass NULL keyboard layout to IntToUnicodeEx

Browse source 
There's an ASSERT on that
This commit is contained in:
Jérôme Gardou 2021-06-18 12:43:53 +02:00 committed by Jérôme Gardou
parent 8d665f8959
commit 813d9cd2cc
1 changed files with 28 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

37
win32ss/user/ntuser/keyboard.c
View file

@ -1105,7 +1105,7 @@ UserProcessKeyboardInput(
if (wVk & KBDEXT)
KbdInput.dwFlags |= KEYEVENTF_EXTENDEDKEY;
//
// Based on wine input:test_Input_blackbox this is okay. It seems the
// Based on wine input:test_Input_blackbox this is okay. It seems the
// bit did not get set and more research is needed. Now the right
// shift works.
//
@ -1341,6 +1341,7 @@ NtUserToUnicodeEx(
PWCHAR pwszBuff = NULL;
INT i, iRet = 0;
PKL pKl = NULL;
NTSTATUS Status;
TRACE("Enter NtUserSetKeyboardState\n");
@ -1390,17 +1391,35 @@ NtUserToUnicodeEx(
pKl = pti->KeyboardLayout;
}
iRet = IntToUnicodeEx(wVirtKey,
wScanCode,
afKeyState,
pwszBuff,
cchBuff,
wFlags,
pKl ? pKl->spkf->pKbdTbl : NULL);
if (pKl)
{
iRet = IntToUnicodeEx(wVirtKey,
wScanCode,
afKeyState,
pwszBuff,
cchBuff,
wFlags,
pKl->spkf->pKbdTbl);
if (iRet)
{
Status = MmCopyToCaller(pwszBuffUnsafe, pwszBuff, cchBuff * sizeof(WCHAR));
}
}
else
{
ERR("No keyboard layout ?!\n");
Status = STATUS_INVALID_HANDLE;
}
MmCopyToCaller(pwszBuffUnsafe, pwszBuff, cchBuff * sizeof(WCHAR));
ExFreePoolWithTag(pwszBuff, TAG_STRING);
if (!NT_SUCCESS(Status))
{
iRet = 0;
SetLastNtError(Status);
}
UserLeave();
TRACE("Leave NtUserSetKeyboardState, ret=%i\n", iRet);
return iRet;