wolfssl icon indicating copy to clipboard operation
wolfssl copied to clipboard

Integrate fuzz targets

Open levirak opened this issue 7 years ago • 3 comments

Here's a pull request that I'm pretty sure accomplishes everything we talked about on the Monday team call. Hopefully my code is all up to standard.

levirak avatar Aug 09 '17 22:08 levirak

I suppose a bit more of a comment is warranted. This integrates the OSS-Fuzz-targets repository into wolfSSL as a subdirectory. This allows us to implement a thin front end (fuzz/target_main.c in this PR) to run the fuzz targets for regression testing. It also has the side effect of making Google's OSS-Fuzz team happier with us and our integration.

On my end, all of the changes I've made don't break make check or make distcheck. In fact, it adds a script, fuzz/all_targets.test, to run during make check, and that seems to do fine.

I've never had to deal with the autotools before, so I'm interested to hear if there is a better/more robust way to do what I've done in configure.ac and fuzz/**/include.am.

levirak avatar Aug 09 '17 22:08 levirak

Final review to be done by Todd Ouska before merging.

kaleb-himes avatar Aug 16 '17 23:08 kaleb-himes

Can one of the wolfSSL admins verify this patch?

wolfSSL-Bot avatar Sep 17 '21 15:09 wolfSSL-Bot

closing, not really relevant any more. We have external oss-fuzz tie in with cryptofuzz https://github.com/guidovranken/cryptofuzz and internal repo with additional fuzzing

JacobBarthelmeh avatar Dec 28 '22 23:12 JacobBarthelmeh