curl / curl-fuzzer

Quality assurance testing for the curl project
MIT License
82 stars 29 forks source link

fuzz_url.cc: fuzz the URL parser #59

Closed bagder closed 1 year ago

bagder commented 1 year ago

What else do we need to fuzz it?

bagder commented 1 year ago

I assume maybe we should somehow change the fourth argument's bitmask as well for better coverage.

cmeister2 commented 1 year ago

You need to add a directory in corpora/ called fuzz_url with at least one test-case file in it otherwise make check can run forever, IIRC.

cmeister2 commented 1 year ago

You also need to add this new fuzzer to scripts/fuzz_targets.

cmeister2 commented 1 year ago

Other than those comments, the actual function of the fuzzer looks good!

bagder commented 1 year ago

@cmeister2 think we can merge this and see what happens?