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

test: Add UBSan -fsanitize=integer suppressions for src/secp256k1 subtree #28131

Merged
merged 1 commit into from Aug 1, 2023

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented Jul 23, 2023

Required for #27991 (see the comment) and for the upcoming CMake-based build system.

@DrahtBot
Copy link
Contributor

DrahtBot commented Jul 23, 2023

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK MarcoFalke
Concept ACK dergoegge

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

@DrahtBot DrahtBot changed the title Add UBSan -fsanitize=integer suppressions for src/secp256k1 subtree test: Add UBSan -fsanitize=integer suppressions for src/secp256k1 subtree Jul 24, 2023
@DrahtBot DrahtBot added the Tests label Jul 24, 2023
@dergoegge
Copy link
Member

Concept ACK

@maflcko
Copy link
Member

maflcko commented Jul 31, 2023

lgtm ACK a747774

@fanquake fanquake merged commit fadad10 into bitcoin:master Aug 1, 2023
15 checks passed
@hebasto hebasto deleted the 230723-ubsan branch August 1, 2023 09:08
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants