Skip to content

Instrument std::string with ASAN contiguous container annotations. #26754

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

Closed
llvmbot opened this issue Jan 30, 2016 · 2 comments · Fixed by #72677
Closed

Instrument std::string with ASAN contiguous container annotations. #26754

llvmbot opened this issue Jan 30, 2016 · 2 comments · Fixed by #72677
Assignees
Labels
bugzilla Issues migrated from bugzilla confirmed Verified by a second party libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.

Comments

@llvmbot
Copy link
Member

llvmbot commented Jan 30, 2016

Bugzilla Link 26380
Version unspecified
OS Linux
Reporter LLVM Bugzilla Contributor
CC @mclow

Extended Description

__sanitizer_annotate_contiguous_container clearly mark used/unused portions of container storage and help catch out of bounds errors.

@llvmbot
Copy link
Member Author

llvmbot commented Jan 30, 2016

Proposed implementation: http://reviews.llvm.org/D12577

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 10, 2021
@llvmbot llvmbot added the confirmed Verified by a second party label Jan 26, 2022
@AdvenamTacet
Copy link
Member

There is a new, independent implementation, currently split into three revisions:

That implementation is already somehow tested, but after that change it is required to build a separate libc++ dylibs with ASan, so we are waiting with landing for #33347 (we want to ship those dylibs with LLVM and we want driver to choose correct ones automatically, that should help with avoiding compatibility issues after that change).
While it is change "breaking ABI", ASan ABI is not stable.

@AdvenamTacet AdvenamTacet linked a pull request Dec 1, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla confirmed Verified by a second party libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants