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

Unit tests for ASan runtime need to be statically linked with asan_*.o files #222

Closed
ramosian-glider opened this issue Aug 31, 2015 · 2 comments

Comments

@ramosian-glider
Copy link
Member

Originally reported on Google Code with ID 222

Currently there's a single test binary that is used for both functional tests for the
-fsanitize=address compiler flag (see ASAN_INST_TEST_SOURCES in asan/tests/CMakeLists.txt)
and unit tests of the ASan runtime (ASAN_NOINST_TEST_SOURCES).

Unit tests need to invoke private functions from the ASan runtime, which are hidden
in the dynamic runtime library on OSX (and probably Android). Therefore it's impossible
to build them in a way similar to the functional tests, unless we mark those private
functions as interfaces.

This isn't a good approach to testing, so instead we must split the unit tests from
those that require instrumentation and statically link the unit tests with the ASan
runtime implementation.

Reported by ramosian.glider on 2013-09-12 09:46:56

@ramosian-glider
Copy link
Member Author

This is now fixed (at r190860).

Reported by samsonov@google.com on 2013-09-17 09:45:57

  • Status changed: Fixed

@ramosian-glider
Copy link
Member Author

Adding Project:AddressSanitizer as part of GitHub migration.

Reported by ramosian.glider on 2015-07-30 09:13:42

  • Labels added: ProjectAddressSanitizer

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

1 participant