tests: Add AddressSanitizer (ASan) Travis build #14794
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
maflcko
reviewed
Nov 23, 2018
Member
|
Any reason to not enable ASan and UBSan in the same test? |
practicalswift
force-pushed
the
asan-in-travis
branch
3 times, most recently
from
7fdb0a7 to
7e5e3fa
Compare
maflcko
reviewed
Nov 25, 2018
practicalswift
force-pushed
the
asan-in-travis
branch
7 times, most recently
from
64b8928 to
0a88263
Compare
Contributor
Author
|
@sipa I was afraid the combined runtime would be too long, but it appears to work. The two Travis jobs have now been merged. Please review. @MarcoFalke Now running also with LSan (LeakSanitizer). Please review :-) |
Member
|
Could squash everything except the changes in test/functional? |
practicalswift
changed the title
practicalswift
force-pushed
the
asan-in-travis
branch
from
0a88263 to
20a6769
Compare
Contributor
Author
|
@MarcoFalke Done! Please re-review :-) |
maflcko
approved these changes
Nov 26, 2018
maflcko
changed the title
Contributor
Author
|
@MarcoFalke Added comments as suggested. Please re-review :-) |
practicalswift
force-pushed
the
asan-in-travis
branch
from
388203a to
1a819f8
Compare
practicalswift
force-pushed
the
asan-in-travis
branch
from
1a819f8 to
3f9b824
Compare
maflcko
previously requested changes
Nov 26, 2018
practicalswift
force-pushed
the
asan-in-travis
branch
2 times, most recently
from
78a62b6 to
65b019a
Compare
Munkybooty
pushed a commit
to Munkybooty/dash
that referenced
this pull request
May 23, 2022
6541d59 Add LSan suppression warnings (practicalswift) ff7212e Add ASan Travis build (practicalswift) ebd3bf2 Make test p2p_invalid_messages.py pass: Allow for expected Travis ASAN memory increase (practicalswift) Pull request description: Add ASan Travis build. Tree-SHA512: b9712aaf0c9112b637b6ef0c5d93961863dcbecaf31d9561eb09258a61540fb31d2c8ecae86518a82763279e4aa6cac266cd352c2b2507df0335c0199f8b3d78
kwvg
added a commit
to kwvg/dash
that referenced
this pull request
Jul 1, 2022
UdjinM6
added a commit
to dashpay/dash
that referenced
this pull request
Jul 2, 2022
backport: bitcoin#16597, bitcoin#17176, bitcoin#14794, bitcoin#17205, bitcoin#17240, bitcoin#17367, bitcoin#18862, bitcoin#21405 (ci reworking: part 2)
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add ASan Travis build.