search

rust-fuzz / targets

🎯 A collection of fuzzing targets written in Rust.
Creative Commons Zero v1.0 Universal
104 stars 21 forks source link

Add a Rust CLI #103

Closed killercup closed 6 years ago

killercup commented 6 years ago

r? @PaulGrandperrin

killercup commented 6 years ago

As a next step we should add a way to easily collect the results of the fuzzers in a convenient way.

killercup commented 6 years ago

Oh: We could rename the run subcommands a bit (mainly "run" to "target") and get rid of the -- in the docs (cargo supports this):

$ cargo run target
$ cargo run continuously
$ cargo run list-targets

Yeah, I'll go ahead with this.

PaulGrandperrin commented 6 years ago

I can't really review the structopt DSL as I have no experience with it but from a user perspective I think it works great! I think the only feature "lost in translation" is the ability to put weights on each targets, but it can be done later. You can merge as soon as you want, I'd like to start doing something about the "compile all targets at once" problem.

killercup commented 6 years ago

Either of you: Feel free to merge, we can easily add more stuff in follow up PRs.

I think the only feature "lost in translation" is the ability to put weights on each targets, but it can be done later.

Ah yes, I explicitly skipped this feature for now. I thought I had commented on that in a commit message, but it seems I forgot. Sorry about that.

What was you main goal with that? To make sure expensive fuzzers get some more time to get through a useful number of iterations?

I like the fact that now this list is automatically extracted from the code

It's currently done in quite a hack-y way (regex), and I'm not sure how best to add a multiplier there.

By the way, I was surprised to see there was no timeout to stop after ≥x seconds of no new branch found, by the way.

frewsxcv commented 6 years ago

bors r+

🎉🎉🎉🎉🎉🎉

bors[bot] commented 6 years ago

Build succeeded