Differential D15140

fuzz: Configure check for main function
ClosedPublic
Actions

Authored by Fabien on Jan 11 2024, 20:52.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCdb55c8d2cc09: fuzz: Configure check for main function

Summary

This determines at cmake time if the main needs to be provided rather than relying on the compiler preprocessor definitions.

Backport of core#20065 and core#21080.

Test Plan

cmake -GNinja .. -DCMAKE_C_COMPILER=afl-gcc \
-DCMAKE_CXX_COMPILER=afl-g++ \
-DCMAKE_BUILD_WITH_INSTALL_RPATH=ON

Check in the cmake output that all the fuzz target need a main()

ninja bitcoin-fuzzer

cmake -GNinja .. -DCMAKE_C_COMPILER=clang \
-DCMAKE_CXX_COMPILER=clang++ \
-DENABLE_SANITIZERS="fuzzer;address;undefined"

Check in the cmake output that no fuzz target need a main()

ninja bitcoin-fuzzer

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Jan 11 2024, 20:52

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 11 2024, 20:52

Fabien requested review of this revision.Jan 11 2024, 20:52

teamcity edited the summary of this revision. (Show Details)Jan 11 2024, 20:52

Harbormaster completed remote builds in B26275: Diff 44074.Jan 11 2024, 21:09

PiRK accepted this revision.Jan 12 2024, 07:39

This revision is now accepted and ready to land.Jan 12 2024, 07:39

Closed by commit rABCdb55c8d2cc09: fuzz: Configure check for main function (authored by Fabien). · Explain WhyJan 12 2024, 13:43

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCdb55c8d2cc09: fuzz: Configure check for main function.

Revision Contents
Changeset List

Path

Size

src/

test/

fuzz/

CMakeLists.txt

23 lines

fuzz.cpp

9 lines

Diff 44090

View Options

src/test/fuzz/CMakeLists.txt