Skip to content

build: Disallow building fuzz binary without -DBUILD_FOR_FUZZING

Placeholder github Source User requested to merge github/fork/dollarparity/patch-1 into master

This pull request addresses issue #31057.

Build System:

  • Modified CMakeLists.txt to prevent building the fuzz binary unless BUILD_FOR_FUZZING is enabled.
  • Added a fatal error message if an attempt is made to build the fuzz binary without BUILD_FOR_FUZZING.

Building the fuzz binary without -DBUILD_FOR_FUZZING results in a binary that is less effective for testing because it won't crash on Assume statements and won't bypass fuzz blockers with FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION.

By enforcing -DBUILD_FOR_FUZZING, we ensure that:

  • The fuzz binary is correctly configured for fuzz testing.
  • Compile failures are caught early in the CI on all platforms.
  • Workarounds like #31028 are no longer necessary.

Merge request reports