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

fuzz: Add fuzz for askama_parser #862

Merged
merged 1 commit into from
Sep 13, 2023
Merged

fuzz: Add fuzz for askama_parser #862

merged 1 commit into from
Sep 13, 2023

Conversation

manunio
Copy link
Contributor

@manunio manunio commented Sep 11, 2023

Hi, I would like to help integrate this project into OSS-Fuzz.

  • As an initial step for integration I have created this PR: askama: initial integration google/oss-fuzz#10935, it contains necessary logic from an OSS-Fuzz perspective to integrate askama.

  • OSS-Fuzz is a free service run by Google that performs continuous fuzzing of important open source projects, you can find more details in our FAQ.

Note:

  • This fuzz_target was able to find bug Fuzzing askama_parser results in panic #860
  • I have limited input len to 500 as anything greater than that was resulting in stack-overflow which was crashing fuzz_target repeatedly.

@djc djc merged commit 8f3140a into djc:main Sep 13, 2023
18 checks passed
@manunio manunio deleted the add-fuzz branch September 13, 2023 14:11
manunio added a commit to manunio/askama that referenced this pull request Sep 18, 2023
While working on djc#862 fuzz_parser was
crashing and failing oss-fuzz ci. so a limit of 500 was placed, this pr removes that.
manunio added a commit to manunio/askama that referenced this pull request Sep 18, 2023
While working on djc#862 fuzz_parser was
crashing and failing oss-fuzz build_checks. so a limit of 500 was placed, this pr removes that.
@manunio manunio mentioned this pull request Sep 18, 2023
djc pushed a commit that referenced this pull request Sep 18, 2023
While working on #862 fuzz_parser was
crashing and failing oss-fuzz build_checks. so a limit of 500 was placed, this pr removes that.
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