Should fix https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=62455
fuzz: Add missing PROVIDE_FUZZ_MAIN_FUNCTION guard to __AFL_FUZZ_INIT #28506
pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2309-fuzz-fix- changing 1 files +1 −1-
maflcko commented at 11:42 AM on September 19, 2023: member
-
fuzz: Add missing PROVIDE_FUZZ_MAIN_FUNCTION guard to __AFL_FUZZ_INIT fa33b2c889
-
DrahtBot commented at 11:42 AM on September 19, 2023: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers 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 added the label Tests on Sep 19, 2023
-
maflcko commented at 1:44 PM on September 19, 2023: member
Locally it failed once for a reason I forgot, but then the re-run passed. @dergoegge rfm?
- dergoegge approved
-
dergoegge commented at 1:56 PM on September 19, 2023: member
utACK fa33b2c889b90bd44f188ba5f0fafe31d7d7fad7
This should fix it
-
maflcko commented at 1:58 PM on September 19, 2023: member
For reference, the failure was in cryptofuzz:
/src/aflplusplus/afl-clang-fast++ -fstack-protector -m64 -pthread -O1 -fno-omit-frame-pointer -gline-tables-only -DFUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION -fsanitize=address -fsanitize-address-use-after-scope -stdlib=libc++ -DCRYPTOFUZZ_NO_OPENSSL -I /src/boost_1_74_0/ -DCRYPTOFUZZ_SECP256K1 -DCRYPTOFUZZ_TREZOR_FIRMWARE -std=c++20 -D_REENTRANT -O1 -fno-omit-frame-pointer -gline-tables-only -DFUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION -fsanitize=address -fsanitize-address-use-after-scope -stdlib=libc++ -DCRYPTOFUZZ_NO_OPENSSL -I /src/boost_1_74_0/ -DCRYPTOFUZZ_SECP256K1 -DCRYPTOFUZZ_TREZOR_FIRMWARE -DBOTAN_IS_BEING_BUILT -Wall -Wextra -Wpedantic -Wshadow -Wstrict-aliasing -Wstrict-overflow=5 -Wcast-align -Wmissing-declarations -Wpointer-arith -Wcast-qual -Wshorten-64-to-32 -Wcomma -Wdocumentation -I build/include -isystem build/include/external -c src/lib/pubkey/sphincsplus/sphincsplus_common/sp_treehash.cpp -o build/obj/lib/pubkey_sphincsplus_common_sp_treehash.o ERROR:__main__:Building fuzzers failed. - fanquake merged this on Sep 19, 2023
- fanquake closed this on Sep 19, 2023
- maflcko deleted the branch on Sep 19, 2023
- Frank-GER referenced this in commit 16aa2e6d6c on Sep 25, 2023
- sidhujag referenced this in commit d2ada08cfe on Sep 26, 2023
- bitcoin locked this on Sep 18, 2024