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

Enable address sanitizer in CI #5053

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

justinchuby
Copy link
Contributor

No description provided.

@justinchuby justinchuby requested a review from a team as a code owner March 28, 2023 05:23
Signed-off-by: Justin Chu <justinchu@microsoft.com>
@justinchuby
Copy link
Contributor Author

==3231==ASan runtime does not come first in initial library list; you should either link runtime to your application or manually preload it with LD_PRELOAD.

@justinchuby
Copy link
Contributor Author

@jcwchen I remember you were able to link asan successfully? I am having a hard time doing that

Copy link
Member

@jcwchen jcwchen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jcwchen I remember you were able to link asan successfully? I am having a hard time doing that

I have tried ASAN for a while, but never succeeded... That's why I only enabled UBSan in my PR to faster unblock old issue.

@justinchuby justinchuby marked this pull request as draft April 5, 2023 18:06
Signed-off-by: Justin Chu <justinchuby@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants