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

wasmtime: Remove usage of initialized corpus #12993

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

alexcrichton
Copy link
Contributor

Wasmtime's initial corpus repository hasn't been updated in years. Many of its files are entirely unused at this time. Other preexisting corpus files no longer test what they originally did as Wasmtime's fuzzing relies on interpreting the input as a "DNA string" where the exact meaning of the DNA changes over time as we change fuzz targets. We concluded recently in Wasmtime to archive/delete the repository we were previously using to avoid confusion so this change follows-up on the OSS-Fuzz side of thing to remove usage of this outdated corpus.

cc @fitzgen

Wasmtime's initial corpus repository hasn't been updated in years. Many
of its files are entirely unused at this time. Other preexisting corpus
files no longer test what they originally did as Wasmtime's fuzzing
relies on interpreting the input as a "DNA string" where the exact
meaning of the DNA changes over time as we change fuzz targets. We
concluded recently in Wasmtime to archive/delete the repository we were
previously using to avoid confusion so this change follows-up on the
OSS-Fuzz side of thing to remove usage of this outdated corpus.
Copy link

alexcrichton is either the primary contact or is in the CCs list of projects/wasmtime.
alexcrichton has previously contributed to projects/wasmtime. The previous PR was #12947

@DavidKorczynski DavidKorczynski merged commit 6dd695a into google:master Jan 31, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants