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

OSS Fuzz Integration #94

Merged
merged 3 commits into from
Sep 3, 2024
Merged

Conversation

ennamarie19
Copy link
Contributor

Hi Gram,
My apologies for the delay on this but here are my fuzz harness and build script for your project! This Pull-Request includes the necessary changes to integrate fuzzing into textdistance for OSS-Fuzz continuous fuzzing, as discussed in the email chain from May 29, 2024.

In short, this PR adds atheris (the fuzzing framework) as a development dependency, an initial harness, the necessary CI file to integrate the project into OSS-Fuzz, and a build script to be used by ClusterFuzz to prepare for nightly fuzzing.

NOTE: The CIFuzz.yml job will fail until Google merges the necessary textdistance Dockerfile into their OSS-Fuzz repository. This can only be done after this PR is merged.

@orsinium
Copy link
Member

orsinium commented Sep 3, 2024

Thank you! There are a few things that we could improve a bit, but nothing big. Let's roll with it :)

@orsinium orsinium merged commit 9cf4a14 into life4:master Sep 3, 2024
9 of 11 checks passed
DonggeLiu pushed a commit to google/oss-fuzz that referenced this pull request Oct 1, 2024
This pull requests integrates the Dockerfile needed to build the fuzzers
for textdistance, as merged into upstream here -
life4/textdistance#94
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants