Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

freetype_ftfuzzer oss-fuzz benchmark uses outdated commit #1925

Open
ndrewh opened this issue Dec 6, 2023 · 0 comments
Open

freetype_ftfuzzer oss-fuzz benchmark uses outdated commit #1925

ndrewh opened this issue Dec 6, 2023 · 0 comments

Comments

@ndrewh
Copy link
Contributor

ndrewh commented Dec 6, 2023

the benchmark.yaml in master currently is quite misleading:

commit: cd02d359a6d0455e9d16b87bf9665961c4699538
commit_date: 2023-01-28T16:04:38+00:00
fuzz_target: ftfuzzer
project: freetype2

That commit (cd02d359a6d0455e9d16b87bf9665961c4699538) is not from 2023, it is from 2017. Imagine my surprise when a long fuzzing run ended and I look through the coverage report and find out I'm looking at 6-year-old code!

As far as the freetype build issue referenced in #1688 goes, it looks like someone is not setting or using SANITIZER correctly. If you add export SANITIZER=none to the build.sh script that is copied in when you run:

python3 benchmarks/oss_fuzz_benchmark_integration.py -p freetype2 -f ftfuzzer -n ftfuzzer_01f23 -c 01f23c39e01d2c739fd8abc69aff9dbf13b46202 -d 2023-12-02T04:13:28+00:00

it builds fine. I didn't look into it further, and this probably isn't an ideal solution since presumably some fuzzers want sanitizers in some contexts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant