The VM layout is not stable between iOS version releases, so switch to dynamic shadow offset.
Details
- Reviewers
kcc etienneb kubamracek - Commits
- rGb17a5db2ee46: [asan] Reapply: Switch to using dynamic shadow offset on iOS
rG11ee532c3a55: [asan] Switch to using dynamic shadow offset on iOS
rCRT283375: [asan] Reapply: Switch to using dynamic shadow offset on iOS
rCRT283240: [asan] Switch to using dynamic shadow offset on iOS
rL283375: [asan] Reapply: Switch to using dynamic shadow offset on iOS
rL283240: [asan] Switch to using dynamic shadow offset on iOS
Diff Detail
Event Timeline
LGTM
Does it deserve any kind of test?
lib/sanitizer_common/sanitizer_mac.cc | ||
---|---|---|
753 | if() instead of #if, please |
Does it deserve any kind of test?
We have internal buildbots testing a majority of asan tests on iOS. With this patch, dynamic offset will be the default and all tests should still pass. (Unfortunately, our iOS test scripts will not be usable for external developers, so we keep them private.)
Dynamic shadow also works on macOS. However, to test it in CI, we'd need to add an option to TSan runtime that would force to always use dynamic shadow. It would only work on the mac or Windows. Since dynamic shadow is not needed on the mac, so I am not sure it's worth adding the complexity.
Hi Anna,
This change broke one of Chromium's Mac buildbots because the mach_vm.h header is not available.
https://build.chromium.org/p/chromium.fyi/builders/ClangToTMac/builds/11485/steps/gclient%20runhooks/logs/stdio
Do you know what the status of that header is? Is it publically available?
if() instead of #if, please