Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Assembly instrumentation fails on Android-x86 #353

Open
ramosian-glider opened this issue Sep 1, 2015 · 6 comments
Open

Assembly instrumentation fails on Android-x86 #353

ramosian-glider opened this issue Sep 1, 2015 · 6 comments

Comments

@ramosian-glider
Copy link
Member

Originally reported on Google Code with ID 353

http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/13369/steps/run%20asan%20tests%20%5BAndroid%2Fx86-K%5D/logs/stdio

We've got hardcoded shadow mapping in asm instrumentation code.
Android uses zero-based shadow, unlike Linux.

We should share and reuse AddressSanitizerPass's knowledge about shadow mappings on
various platforms.

Reported by eugenis@google.com on 2014-10-10 12:23:14

@ramosian-glider
Copy link
Member Author

The url in the original message does not work anymore, here is a recent more recent
run of the same test: http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/14766/steps/run%20asan%20tests%20%5BAndroid%2Fx86-K%5D/logs/stdio

Reported by krasin@google.com on 2014-12-20 03:21:35

@ramosian-glider
Copy link
Member Author

Adding Project:AddressSanitizer as part of GitHub migration.

Reported by ramosian.glider on 2015-07-30 09:06:34

  • Labels added: ProjectAddressSanitizer

@eugenis
Copy link
Contributor

eugenis commented Sep 2, 2015

More problems.

  • Hardcoded shadow mapping.
  • cmake in asan/tests checks for CAN_TARGET_I386, but android is I686
  • instrumentation of rep movs uses EBX as a scratch register. On x86 EBX is a PIC register. On Android we link ASan runtime as a shared library, and crash when entering the PLT stub for __asan_report_load with bad EBX.

@eugenis
Copy link
Contributor

eugenis commented Sep 3, 2015

And we are out of registers.

@morehouse morehouse assigned eugenis and unassigned google Jun 6, 2018
@morehouse
Copy link
Contributor

@eugenis: This bug is over 3 years old. Do we want to keep it open?

@eugenis
Copy link
Contributor

eugenis commented Jun 8, 2018

I think we want to deprecate the whole assembly instrumentation thing.
It never evolved into anything useful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants