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

Add ignorelist for ASAN #392

Open
alanking opened this issue Dec 13, 2024 · 2 comments
Open

Add ignorelist for ASAN #392

alanking opened this issue Dec 13, 2024 · 2 comments
Labels
build/tools enhancement New feature or request
Milestone

Comments

@alanking
Copy link

Feature

UBSAN support is being added with #390 and will include an ignorelist in order to exclude third party code from sanitation reports. ASAN also supports ignorelists, so we should add support for that in the same manner.

@alanking alanking added enhancement New feature or request build/tools labels Dec 13, 2024
@alanking alanking added this to the 0.6.0 milestone Dec 13, 2024
@MartinFlores751
Copy link
Contributor

To put a bit more words, all sanitizers use the -fsanitize-ignorelist flag. If we wish to add ignores for ASAN, we would move the flag to the shared flags that are set for the sanitizers (to make things more obvious), and add them to the existing file (for HTTP API at least).

Or use separate ignore files, but that might make things a bit confusing?

Link to ignore list docs: https://releases.llvm.org/13.0.1/tools/clang/docs/SanitizerSpecialCaseList.html

@MartinFlores751
Copy link
Contributor

Another thing to consider for the ignore list (suggested by @SwooshyCueb, see #369 (comment)), use src:/opt/irods-externals/* to ignore all externals.

Relevant here since it would affect all sanitizers used if not scoped.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build/tools enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants