million/suyu
1
0
Fork 0
forked from suyu/suyu
Code Pull requests Activity

gdbstub_arch: Directly access SP register

Browse source 
Currently to access the SP register, RegRead and RegWrite rely on a
out-of-bounds array access to reach the next element in a struct. As
of writing only git versions of GCC catch this error.

Specify the SP register when we want to access it in these functions.
This commit is contained in:
lat9nq 2022-06-30 12:40:01 -04:00
parent 01bc0c84f0
commit 7b0affb6e0
1 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/core/debugger/gdbstub_arch.cpp
View file

@ -191,8 +191,10 @@ std::string GDBStubA64::RegRead(const Kernel::KThread* thread, size_t id) const
const auto& gprs{context.cpu_registers}; const auto& gprs{context.cpu_registers};
const auto& fprs{context.vector_registers}; const auto& fprs{context.vector_registers};
if (id <= SP_REGISTER) { if (id < SP_REGISTER) {
return ValueToHex(gprs[id]); return ValueToHex(gprs[id]);
} else if (id == SP_REGISTER) {
return ValueToHex(context.sp);
} else if (id == PC_REGISTER) { } else if (id == PC_REGISTER) {
return ValueToHex(context.pc); return ValueToHex(context.pc);
} else if (id == PSTATE_REGISTER) { } else if (id == PSTATE_REGISTER) {
@ -215,8 +217,10 @@ void GDBStubA64::RegWrite(Kernel::KThread* thread, size_t id, std::string_view v
auto& context{thread->GetContext64()}; auto& context{thread->GetContext64()};
if (id <= SP_REGISTER) { if (id < SP_REGISTER) {
context.cpu_registers[id] = HexToValue<u64>(value); context.cpu_registers[id] = HexToValue<u64>(value);
} else if (id == SP_REGISTER) {
context.sp = HexToValue<u64>(value);
} else if (id == PC_REGISTER) { } else if (id == PC_REGISTER) {
context.pc = HexToValue<u64>(value); context.pc = HexToValue<u64>(value);
} else if (id == PSTATE_REGISTER) { } else if (id == PSTATE_REGISTER) {