From: Alan Hayward Date: Fri, 22 Mar 2019 10:39:11 +0000 (+0000) Subject: AArch64: DWARF unwinder support for signed return addresses X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=11e1b75f0696f6e406fe8ccc759bac2fbc8fc151;p=binutils-gdb.git AArch64: DWARF unwinder support for signed return addresses Pauth address signing is enabled at binary compile time. When enabled the return addresses for functions may be mangled. This patch adds functionality to restore the original address for use in the DWARF unwinder. DW_CFA_AARCH64_negate_ra_state in a binary indicates the toggling of address signing between enabled and disabled. Ensure the state is stored in the DWARF register ra_state. Ensure the pauth DWARF registers are initialised. gdb/ChangeLog: * aarch64-tdep.c (aarch64_frame_unmask_address): New function. (aarch64_dwarf2_prev_register): Unmask PC value. (aarch64_dwarf2_frame_init_reg): Init pauth registers. (aarch64_execute_dwarf_cfa_vendor_op): Check for DW_CFA_AARCH64_negate_ra_state. (aarch64_gdbarch_init): Add aarch64_execute_dwarf_cfa_vendor_op. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 25c43f0faa5..13b117205df 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,13 @@ +2019-03-22 Alan Hayward + Jiong Wang + + * aarch64-tdep.c (aarch64_frame_unmask_address): New function. + (aarch64_dwarf2_prev_register): Unmask PC value. + (aarch64_dwarf2_frame_init_reg): Init pauth registers. + (aarch64_execute_dwarf_cfa_vendor_op): Check for + DW_CFA_AARCH64_negate_ra_state. + (aarch64_gdbarch_init): Add aarch64_execute_dwarf_cfa_vendor_op. + 2019-03-22 Alan Hayward Jiong Wang diff --git a/gdb/aarch64-tdep.c b/gdb/aarch64-tdep.c index 01e98b7195e..07430c0f25b 100644 --- a/gdb/aarch64-tdep.c +++ b/gdb/aarch64-tdep.c @@ -34,6 +34,7 @@ #include "frame-base.h" #include "trad-frame.h" #include "objfiles.h" +#include "dwarf2.h" #include "dwarf2-frame.h" #include "gdbtypes.h" #include "prologue-value.h" @@ -248,6 +249,26 @@ class instruction_reader : public abstract_instruction_reader } // namespace +/* If address signing is enabled, mask off the signature bits from ADDR, using + the register values in THIS_FRAME. */ + +static CORE_ADDR +aarch64_frame_unmask_address (struct gdbarch_tdep *tdep, + struct frame_info *this_frame, + CORE_ADDR addr) +{ + if (tdep->has_pauth () + && frame_unwind_register_unsigned (this_frame, + tdep->pauth_ra_state_regnum)) + { + int cmask_num = AARCH64_PAUTH_CMASK_REGNUM (tdep->pauth_reg_base); + CORE_ADDR cmask = frame_unwind_register_unsigned (this_frame, cmask_num); + addr = addr & ~cmask; + } + + return addr; +} + /* Analyze a prologue, looking for a recognizable stack frame and frame pointer. Scan until we encounter a store that could clobber the stack frame unexpectedly, or an unknown instruction. */ @@ -1013,12 +1034,14 @@ static struct value * aarch64_dwarf2_prev_register (struct frame_info *this_frame, void **this_cache, int regnum) { + struct gdbarch_tdep *tdep = gdbarch_tdep (get_frame_arch (this_frame)); CORE_ADDR lr; switch (regnum) { case AARCH64_PC_REGNUM: lr = frame_unwind_register_unsigned (this_frame, AARCH64_LR_REGNUM); + lr = aarch64_frame_unmask_address (tdep, this_frame, lr); return frame_unwind_got_constant (this_frame, regnum, lr); default: @@ -1027,6 +1050,9 @@ aarch64_dwarf2_prev_register (struct frame_info *this_frame, } } +static const unsigned char op_lit0 = DW_OP_lit0; +static const unsigned char op_lit1 = DW_OP_lit1; + /* Implement the "init_reg" dwarf2_frame_ops method. */ static void @@ -1034,18 +1060,72 @@ aarch64_dwarf2_frame_init_reg (struct gdbarch *gdbarch, int regnum, struct dwarf2_frame_state_reg *reg, struct frame_info *this_frame) { + struct gdbarch_tdep *tdep = gdbarch_tdep (gdbarch); + switch (regnum) { case AARCH64_PC_REGNUM: reg->how = DWARF2_FRAME_REG_FN; reg->loc.fn = aarch64_dwarf2_prev_register; - break; + return; + case AARCH64_SP_REGNUM: reg->how = DWARF2_FRAME_REG_CFA; - break; + return; + } + + /* Init pauth registers. */ + if (tdep->has_pauth ()) + { + if (regnum == tdep->pauth_ra_state_regnum) + { + /* Initialize RA_STATE to zero. */ + reg->how = DWARF2_FRAME_REG_SAVED_VAL_EXP; + reg->loc.exp.start = &op_lit0; + reg->loc.exp.len = 1; + return; + } + else if (regnum == AARCH64_PAUTH_DMASK_REGNUM (tdep->pauth_reg_base) + || regnum == AARCH64_PAUTH_CMASK_REGNUM (tdep->pauth_reg_base)) + { + reg->how = DWARF2_FRAME_REG_SAME_VALUE; + return; + } } } +/* Implement the execute_dwarf_cfa_vendor_op method. */ + +static bool +aarch64_execute_dwarf_cfa_vendor_op (struct gdbarch *gdbarch, gdb_byte op, + struct dwarf2_frame_state *fs) +{ + struct gdbarch_tdep *tdep = gdbarch_tdep (gdbarch); + struct dwarf2_frame_state_reg *ra_state; + + if (tdep->has_pauth () && op == DW_CFA_AARCH64_negate_ra_state) + { + /* Allocate RA_STATE column if it's not allocated yet. */ + fs->regs.alloc_regs (AARCH64_DWARF_PAUTH_RA_STATE + 1); + + /* Toggle the status of RA_STATE between 0 and 1. */ + ra_state = &(fs->regs.reg[AARCH64_DWARF_PAUTH_RA_STATE]); + ra_state->how = DWARF2_FRAME_REG_SAVED_VAL_EXP; + + if (ra_state->loc.exp.start == nullptr + || ra_state->loc.exp.start == &op_lit0) + ra_state->loc.exp.start = &op_lit1; + else + ra_state->loc.exp.start = &op_lit0; + + ra_state->loc.exp.len = 1; + + return true; + } + + return false; +} + /* When arguments must be pushed onto the stack, they go on in reverse order. The code below implements a FILO (stack) to do this. */ @@ -3192,6 +3272,9 @@ aarch64_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) gdbarch_init_osabi (info, gdbarch); dwarf2_frame_set_init_reg (gdbarch, aarch64_dwarf2_frame_init_reg); + /* Register DWARF CFA vendor handler. */ + set_gdbarch_execute_dwarf_cfa_vendor_op (gdbarch, + aarch64_execute_dwarf_cfa_vendor_op); /* Add some default predicates. */ frame_unwind_append_unwinder (gdbarch, &aarch64_stub_unwind);