Merge bitcoin#32325: ci: Add missing -Wno-error=array-bounds to valgrind fuzz

fanquake · fanquake · commit 8406a9f4f139 · 2025-04-22T17:02:38.000+01:00
fa653cb ci: Add missing -Wno-error=array-bounds to valgrind fuzz (MarcoFalke) Pull request description: Due to an upstream GCC issue, any debug/fuzz build which aborts on failed assumes will print a false positive array-bounds warning in `src/test/fuzz/txgraph.cpp`. This also affects one CI task. Fix the CI task by ignoring the error for now. Fixes bitcoin#32276 ACKs for top commit: fanquake: ACK fa653cb - checked native fuzz Tree-SHA512: 0f6c5ec8d96e0bf96cd008e2de5db59e528086a67dcb77f3e59a0d83225d880a59e960d65c5bc8b5ae3de9d5d301bfc7737d95c282aa1bcc740a42561f610ca8
diff --git a/ci/test/00_setup_env_native_fuzz_with_valgrind.sh b/ci/test/00_setup_env_native_fuzz_with_valgrind.sh
@@ -17,4 +17,5 @@ export FUZZ_TESTS_CONFIG="--valgrind"
 export GOAL="all"
 export BITCOIN_CONFIG="\
  -DBUILD_FOR_FUZZING=ON \
+ -DCMAKE_CXX_FLAGS='-Wno-error=array-bounds' \
 "

Original file line number	Diff line number	Diff line change
`@@ -17,4 +17,5 @@ export FUZZ_TESTS_CONFIG="--valgrind"`
`17`	`17`	`export GOAL="all"`
`18`	`18`	`export BITCOIN_CONFIG="\`
`19`	`19`	`-DBUILD_FOR_FUZZING=ON \`
	`20`	`+ -DCMAKE_CXX_FLAGS='-Wno-error=array-bounds' \`
`20`	`21`	`"`