Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2025-05-06 18:31:26 +00:00
Code Issues Projects Releases Packages Wiki Activity

[NTOSKRNL/x64] Fix a bug in KeSwitchKernelStack

Browse source 
Don't safe anything in the callee's home space, because the callee can overwrite it. Use the functions home space instead.
This commit is contained in:
Timo Kreuzer 2023-08-15 22:51:05 +03:00
parent d41a4030bc
commit 26a64324e7
1 changed files with 4 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
ntoskrnl/ke/amd64/trap.S
View file

@ -1213,19 +1213,18 @@ EXTERN KiSwitchKernelStack:PROC
PUBLIC KeSwitchKernelStack PUBLIC KeSwitchKernelStack
FUNC KeSwitchKernelStack FUNC KeSwitchKernelStack
/* Save rcx and allocate callee home space */
mov [rsp + P1Home], rcx
.savereg rcx, P1Home
sub rsp, 40 sub rsp, 40
.allocstack 40 .allocstack 40
/* Save rcx */
mov [rsp], rcx
.savereg rcx, 0
.endprolog .endprolog
/* Call the C handler, which returns the old stack in rax */ /* Call the C handler, which returns the old stack in rax */
call KiSwitchKernelStack call KiSwitchKernelStack
/* Restore rcx (StackBase) */ /* Restore rcx (StackBase) */
mov rcx, [rsp] mov rcx, [rsp + 40 + P1Home]
/* Switch to new stack: RSP += (StackBase - OldStackBase) */ /* Switch to new stack: RSP += (StackBase - OldStackBase) */
sub rcx, rax sub rcx, rax