Skip to content

Commit

Permalink
fix: add gofuzz tags to build (#10884)
Browse files Browse the repository at this point in the history
It appears that `gofuzz` was removed from the default tagspec in the
main branch of go-118-fuzz-build [^1] to `gofuzz_libfuzzer` and
`libfuzzer`. To keep compatibility with the go build scripts, this
change adds the gofuzz tag to the go-118-fuzz-build invocation.

[^1]:
https://github.com/AdamKorcz/go-118-fuzz-build/pull/9/files#diff-2873f79a86c0d8b3335cd7731b0ecf7dd4301eb19a82ef7a1cba7589b5252261L35
  • Loading branch information
arirubinstein authored Aug 29, 2023
1 parent 307a03a commit ce908a4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion infra/base-images/base-builder/compile_native_go_fuzzer
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ function build_native_go_fuzzer() {
cp "${fuzzer_filename}" "${OUT}/rawfuzzers/${fuzzer}"
cd $current_dir
else
go-118-fuzz-build -o $fuzzer.a -func $function $abs_file_dir
go-118-fuzz-build $tags -o $fuzzer.a -func $function $abs_file_dir
$CXX $CXXFLAGS $LIB_FUZZING_ENGINE $fuzzer.a -o $OUT/$fuzzer
fi
}
Expand Down

0 comments on commit ce908a4

Please sign in to comment.