diff options
author | Joshua Peraza <jperaza@chromium.org> | 2018-07-06 13:22:17 -0700 |
---|---|---|
committer | Joshua Peraza <jperaza@chromium.org> | 2018-07-06 21:22:45 +0000 |
commit | 89e7a8615f3d39c802ce27c21ce62646f10291ef (patch) | |
tree | 47c8aecd8c92e467b7826e44a89cf19a24e17647 /src/processor/stackwalker_arm_unittest.cc | |
parent | Ignore duplicate module list entries. (diff) | |
download | breakpad-89e7a8615f3d39c802ce27c21ce62646f10291ef.tar.xz |
iOS: disable using CFI in stackwalking
dump_syms produces incomplete CFI info on iOS because it doesn't support
converting compact unwind to Breakpad symbols. Attempting to use
incomplete CFI can result in infinte stack traces.
Bug: google-breakpad:764
Change-Id: Id042aa515d17928cb5503a79038607d95c56238d
Reviewed-on: https://chromium-review.googlesource.com/1128252
Reviewed-by: Ivan Penkov <ivanpe@chromium.org>
Diffstat (limited to 'src/processor/stackwalker_arm_unittest.cc')
-rw-r--r-- | src/processor/stackwalker_arm_unittest.cc | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/processor/stackwalker_arm_unittest.cc b/src/processor/stackwalker_arm_unittest.cc index 8a0fd5e9..2ca7a02a 100644 --- a/src/processor/stackwalker_arm_unittest.cc +++ b/src/processor/stackwalker_arm_unittest.cc @@ -782,8 +782,12 @@ TEST_F(CFI, RejectBadExpressions) { class StackwalkerARMFixtureIOS : public StackwalkerARMFixture { public: StackwalkerARMFixtureIOS() { - system_info.os = "iOS"; - system_info.os_short = "ios"; + // iOS_test is used instead of iOS because the stackwalker has a check to + // avoid using CFI for iOS dumps. This is a workaround for bad CFI being + // produced by dump_syms for iOS. + // https://bugs.chromium.org/p/google-breakpad/issues/detail?id=764 + system_info.os = "iOS_test"; + system_info.os_short = "ios_test"; } }; |