search

m4b / goblin

An impish, cross-platform binary parsing crate, written in Rust
MIT License
1.17k stars 156 forks source link

Add AFL++ fuzzing support #351

Closed anfedotoff closed 1 year ago

anfedotoff commented 1 year ago

Hi! I think adding AFL++ fuzzing support it's a good idea for finding new bugs. What do you think?

m4b commented 1 year ago

well this is annoying looks like our macos CI test is failing (unrelated to your PR) :/

m4b commented 1 year ago

this seems great! (sorry for the delay)

this doesn't add any dependencies to goblin for users right? This is just a crate that is in the project directory, that we can incidentally run fuzzing on, yes?

anfedotoff commented 1 year ago

this seems great! (sorry for the delay)

this doesn't add any dependencies to goblin for users right? This is just a crate that is in the project directory, that we can incidentally run fuzzing on, yes?

Yes it doesn't add any new dependencies. It's like cargo fuzz, but cargo afl with AFL++ inside. You could look at image-rs which have both libFuzzer and AFL++ support.

anfedotoff commented 1 year ago

@m4b, so what do you think, do we need AFL++ fuzzing support?

m4b commented 1 year ago

released in 0.7.0, thank you so much for your patience!