commit 2502a60ea1a5d612ce5d0c44a121fe314c3ef930
author Christopher Ferris <cferris@google.com> Mon Oct 23 13:51:54 2017 -0700
committer Christopher Ferris <cferris@google.com> Mon Oct 23 14:55:16 2017 -0700
tree b8f0f043f8762467120b4c1a70d8a4c0ec5c0811
parent 26b614ddf6a135e37ffd5c2b2a619fdf90569de4

If the pc is set to zero, the unwind is done.

Bug: 68047085

Test: Ran new unit tests, verified new unwinder does not show an
Test: extra pc zero frame for arm 32 bit processes.
Change-Id: Ic6532e56fbb786a8b7d41638abae777c2d0f1d59

libunwindstack/ElfInterfaceArm.cpp

diff --git a/libunwindstack/ElfInterfaceArm.cpp b/libunwindstack/ElfInterfaceArm.cpp
index 30a1532..170a5cd 100644
--- a/libunwindstack/ElfInterfaceArm.cpp
+++ b/libunwindstack/ElfInterfaceArm.cpp
@@ -121,8 +121,10 @@
     }
     regs_arm->set_sp(arm.cfa());
     (*regs_arm)[ARM_REG_SP] = regs_arm->sp();
-    *finished = false;
     return_value = true;
+
+    // If the pc was set to zero, consider this the final frame.
+    *finished = (regs_arm->pc() == 0) ? true : false;
   }
 
   if (arm.status() == ARM_STATUS_NO_UNWIND) {