highperfocused/bitcoin
1
0
Fork 0
mirror of https://github.com/bitcoin/bitcoin.git synced 2026-06-05 18:52:29 +02:00
Code Issues Packages Projects Releases Wiki Activity

ci: Do not patch leveldb to workaround UB in "tidy" CI job

Browse Source
This commit is contained in:
Hennadii Stepanov
2025-09-08 18:32:24 +01:00
parent 314c42b55b
commit 56f2a689a2
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
ci/test/03_test_script.sh
View File

@@ -41,7 +41,10 @@ echo "=== BEGIN env ==="
env env
echo "=== END env ===" echo "=== END env ==="
( # Don't apply patches in the tidy job, because it relies on the `git diff`
# command to detect IWYU errors. It is safe to skip this patch in the tidy job
# because it doesn't run a UB detector.
if [ "$RUN_TIDY" != "true" ]; then
# compact->outputs[i].file_size is uninitialized memory, so reading it is UB. # compact->outputs[i].file_size is uninitialized memory, so reading it is UB.
# The statistic bytes_written is only used for logging, which is disabled in # The statistic bytes_written is only used for logging, which is disabled in
# CI, so as a temporary minimal fix to work around UB and CI failures, leave # CI, so as a temporary minimal fix to work around UB and CI failures, leave
@@ -62,7 +65,7 @@ echo "=== END env ==="
mutex_.Lock(); mutex_.Lock();
stats_[compact->compaction->level() + 1].Add(stats); stats_[compact->compaction->level() + 1].Add(stats);
EOF EOF
) fi
if [ "$RUN_FUZZ_TESTS" = "true" ]; then if [ "$RUN_FUZZ_TESTS" = "true" ]; then
export DIR_FUZZ_IN=${DIR_QA_ASSETS}/fuzz_corpora/ export DIR_FUZZ_IN=${DIR_QA_ASSETS}/fuzz_corpora/