fuzz: add harness for filters and escape

djc / askama

Type-safe, compiled Jinja-like templates for Rust

Apache License 2.0

3.25k stars 213 forks source link

fuzz: add harness for filters and escape #1069

Closed manunio closed 1 month ago

manunio commented 1 month ago

This pr:

Adds a harness for filters and escape, this has helped in finding bugs like https://github.com/djc/askama/pull/1067
Moves all fuzz targets to a root fuzz crate, I think this approach is better than creating it per crate.

manunio commented 1 month ago

Fuzzing failure will be solved once this pr and the https://github.com/google/oss-fuzz/pull/12084 merges. Edit: All checks are green at oss-fuzz so this is safe to merge.

djc commented 1 month ago

Would you mind making this two separate commits (or PRs)? One that moves the existing parser fuzz tests to the new crate, and a separate one that adds the new filter tests.

manunio commented 1 month ago

Would you mind making this two separate commits (or PRs)? One that moves the existing parser fuzz tests to the new crate, and a separate one that adds the new filter tests.

Sure, no problem.