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

test(fuzz): cover other image types #2345

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

LeoDog896
Copy link

I license past and future contributions under the dual MIT/Apache-2.0 license,
allowing licensees to choose either at their option.

Adds fuzz targets for the QOI, OpenEXR, and Farbfeld image formats

Copy link
Contributor

@fintelia fintelia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you checked whether these formats pass fuzzing?

We already have more fuzzing crashes than we have contributor bandwidth to fix, so I don't think wiring up more failing fuzzing harnesses will achieve much

@LeoDog896
Copy link
Author

Have you checked whether these formats pass fuzzing?

I already checked that those formats pass fuzzing for around 20 minutes.

We already have more fuzzing crashes than we have contributor bandwidth to fix, so I don't think wiring up more failing fuzzing harnesses will achieve much

That makes sense, though I doubt these added targets will throw out false positives.

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

Successfully merging this pull request may close these issues.

2 participants