search

mcginty / snow

A Rust implementation of the Noise Protocol Framework
Apache License 2.0
893 stars 119 forks source link

more fuzzers #28

Open mcginty opened 6 years ago

mcginty commented 6 years ago

It'd be great to have an even more robust fuzzer for snow to increase confidence and avoid panics.

stusmall commented 5 years ago

I'd love to take this on. I'll give it a look tonight.

quapka commented 3 years ago

Hi, have you made any progress so far, @stusmall? I might take a look.

stusmall commented 3 years ago

Nothing past the small change that got merged.

quapka commented 3 years ago

Ok, and what is the intended way of running the fuzzer? I've built the hfuzz and tried to running it without a success:

$./target/release/handshake_reader
This executable hasn't been built with "cargo hfuzz".
Try executing "cargo hfuzz build" and check out "hfuzz_target" directory.
Or execute "cargo hfuzz run TARGET"

$ cargo hfuzz
error: no such subcommand: `hfuzz`

$ cargo hfuzz build
error: no such subcommand: `hfuzz`

$ cargo hfuzz run handshake_reader
error: no such subcommand: `hfuzz`
quapka commented 3 years ago

Oh, I see.

$ cargo install honggfuzz

And I also needed sudo apt-get install binutils-dev. Now:

$ cargo hfuzz run handshake_reader

works. I suppose worth adding to the README.md.