diff options
| author |   djm <djm@openbsd.org> | 2015-01-18 19:53:58 +0000 |
|---|---|---|
| committer | djm <djm@openbsd.org> | 2015-01-18 19:53:58 +0000 |
| commit | 57907c7a1a372600a7caf4a84db7eba62aa27706 (patch) | |
| tree | d18994cfc09fe4c52f36123d494f7373b23fc36b /regress/usr.bin/ssh/unittests/sshkey/test_fuzz.c | |
| parent | add a fuzz_matches_original() function to the fuzzer to (diff) | |
| download | wireguard-openbsd-57907c7a1a372600a7caf4a84db7eba62aa27706.tar.xz wireguard-openbsd-57907c7a1a372600a7caf4a84db7eba62aa27706.zip | |
make the signature fuzzing test much more rigorous:
ensure that the fuzzed input cases do not match the
original (using new fuzz_matches_original() function)
and check that the verification fails in each case
Diffstat (limited to 'regress/usr.bin/ssh/unittests/sshkey/test_fuzz.c')
| -rw-r--r-- | regress/usr.bin/ssh/unittests/sshkey/test_fuzz.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/regress/usr.bin/ssh/unittests/sshkey/test_fuzz.c b/regress/usr.bin/ssh/unittests/sshkey/test_fuzz.c index 936a8899cad..0e9cb6f5a78 100644 --- a/regress/usr.bin/ssh/unittests/sshkey/test_fuzz.c +++ b/regress/usr.bin/ssh/unittests/sshkey/test_fuzz.c @@ -1,4 +1,4 @@ -/* $OpenBSD: test_fuzz.c,v 1.1 2014/06/24 01:14:18 djm Exp $ */ +/* $OpenBSD: test_fuzz.c,v 1.2 2015/01/18 19:53:58 djm Exp $ */ /* * Fuzz tests for key parsing * @@ -81,8 +81,11 @@ sig_fuzz(struct sshkey *k) free(sig); TEST_ONERROR(onerror, fuzz); for(; !fuzz_done(fuzz); fuzz_next(fuzz)) { - sshkey_verify(k, fuzz_ptr(fuzz), fuzz_len(fuzz), - c, sizeof(c), 0); + /* Ensure 1-bit difference at least */ + if (fuzz_matches_original(fuzz)) + continue; + ASSERT_INT_NE(sshkey_verify(k, fuzz_ptr(fuzz), fuzz_len(fuzz), + c, sizeof(c), 0), 0); } fuzz_cleanup(fuzz); } |