Skip to content

Commit

Permalink
CI: Temporarily isable MSAN
Browse files Browse the repository at this point in the history
There is a bug in the memory sanitizer at the moment, for more info see

- rust-bitcoin/rust-bitcoin#2600
- rust-bitcoin/rust-bitcoin#2579

As we did in `rust-bitcoin` temporarily disable MSAN in CI.
  • Loading branch information
tcharding committed Mar 26, 2024
1 parent fb676dc commit a7ece9e
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions contrib/_test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -97,9 +97,12 @@ if [ "$DO_ASAN" = true ]; then
cargo clean
# The -Cllvm-args=-msan-eager-checks=0 flag was added to overcome this issue:
# https://github.com/rust-bitcoin/rust-secp256k1/pull/573#issuecomment-1399465995
CC='clang -fsanitize=memory -fno-omit-frame-pointer' \
RUSTFLAGS='-Zsanitizer=memory -Zsanitizer-memory-track-origins -Cforce-frame-pointers=yes -Cllvm-args=-msan-eager-checks=0' \
cargo test --lib --all --features="$FEATURES" -Zbuild-std --target x86_64-unknown-linux-gnu
#
# There is currently a bug in the MemorySanitizer (MSAN) - disable the job for now.
#
# CC='clang -fsanitize=memory -fno-omit-frame-pointer' \
# RUSTFLAGS='-Zsanitizer=memory -Zsanitizer-memory-track-origins -Cforce-frame-pointers=yes -Cllvm-args=-msan-eager-checks=0' \
# cargo test --lib --all --features="$FEATURES" -Zbuild-std --target x86_64-unknown-linux-gnu

pushd "$REPO_DIR/no_std_test" > /dev/null || exit 1
# See https://github.com/rust-bitcoin/rust-secp256k1/pull/641#issuecomment-1671598914
Expand Down

0 comments on commit a7ece9e

Please sign in to comment.