diff options
author | Christopher Ferris <cferris@google.com> | 2018-03-15 14:35:01 -0700 |
---|---|---|
committer | Christopher Ferris <cferris@google.com> | 2018-03-15 15:14:18 -0700 |
commit | 088fd9fa172e9b2463882981f0449ed42fd998c1 (patch) | |
tree | 1a5fe4df82f04627cd83545a00215db84b43633b /libunwindstack/ElfInterfaceArm.h | |
parent | 705374d0dd3533a28a7d7608e97ff6fffd498ed0 (diff) | |
download | unwinding-088fd9fa172e9b2463882981f0449ed42fd998c1.tar.gz |
Handle ARM thumb function symbols.
ARM thumb functions will have the zero bit set to one, which can cause
function name lookups to fail.
Add an ARM special GetFunctionName that handles this condition.
Fix a couple of the function offsets in unit tests.
Bug: 74844137
Test: Ran unit tests.
Test: Ran debuggerd -b on processes on a bullhead device.
Change-Id: Ibd407db34eaaa641f91fdb4f589c44a0dcc0216a
Diffstat (limited to 'libunwindstack/ElfInterfaceArm.h')
-rw-r--r-- | libunwindstack/ElfInterfaceArm.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/libunwindstack/ElfInterfaceArm.h b/libunwindstack/ElfInterfaceArm.h index 9c067ba..c1597ce 100644 --- a/libunwindstack/ElfInterfaceArm.h +++ b/libunwindstack/ElfInterfaceArm.h @@ -76,6 +76,9 @@ class ElfInterfaceArm : public ElfInterface32 { bool StepExidx(uint64_t pc, uint64_t load_bias, Regs* regs, Memory* process_memory, bool* finished); + bool GetFunctionName(uint64_t addr, uint64_t load_bias, std::string* name, + uint64_t* offset) override; + uint64_t start_offset() { return start_offset_; } size_t total_entries() { return total_entries_; } |