ed/raddebugger
1
0
Fork 0
mirror of https://github.com/Ed94/raddebugger.git synced 2026-08-06 05:48:40 +00:00
Code Issues Packages Projects Releases Wiki Activity

ctrl thread stepping machine: read initial stack-pointer-check-value *before* single-stepping stuck threads, otherwise we can obtain an incorrect stack check value if the single-step causes a call.

Browse Source
This commit is contained in:
Ryan Fleury
2024-03-22 17:27:54 -07:00
parent c26274ac70
commit 83ecb4b86a
1 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/ctrl/ctrl_core.c
+9 -1
View File
@@ -2861,6 +2861,15 @@ ctrl_thread__run(DMN_CtrlCtx *ctrl_ctx, CTRL_Msg *msg)
} }
} }
//////////////////////////////
//- rjf: read initial stack-pointer-check value
//
// This MUST happen before any threads move, including single-stepping stuck
// threads, because otherwise, their stack pointer may change, if single-stepping
// causes e.g. entrance into a function via a call instruction.
//
U64 sp_check_value = dmn_rsp_from_thread(target_thread);
////////////////////////////// //////////////////////////////
//- rjf: single step "stuck threads" //- rjf: single step "stuck threads"
// //
@@ -3011,7 +3020,6 @@ ctrl_thread__run(DMN_CtrlCtx *ctrl_ctx, CTRL_Msg *msg)
// //
if(stop_event == 0) if(stop_event == 0)
{ {
U64 sp_check_value = dmn_rsp_from_thread(target_thread);
B32 spoof_mode = 0; B32 spoof_mode = 0;
CTRL_Spoof spoof = {0}; CTRL_Spoof spoof = {0};
for(;;) for(;;)