aboutsummaryrefslogtreecommitdiff
path: root/FUZZING.md
diff options
context:
space:
mode:
authorDavid Benjamin <davidben@google.com>2016-03-21 18:24:53 -0400
committerDavid Benjamin <davidben@google.com>2016-03-22 18:46:35 +0000
commit9539ebbf7042f8eaf8c6ddac25ad01aa4017634e (patch)
tree06f437f7bafb23b824107bb73cc3ea9b3111ff50 /FUZZING.md
parent78f8aabe44da43d03ecf82d8d81f4251cd4bfe4a (diff)
downloadboringssl-9539ebbf7042f8eaf8c6ddac25ad01aa4017634e.zip
boringssl-9539ebbf7042f8eaf8c6ddac25ad01aa4017634e.tar.gz
boringssl-9539ebbf7042f8eaf8c6ddac25ad01aa4017634e.tar.bz2
Update FUZZING documentation about max_len.
Maintain the max_len values in foo.options files which ClusterFuzz can process. Also recompute the recommended client and server lengths as they've since gotten much more extensive. Change-Id: Ie87a80d8a4a0c41e215f0537c8ccf82b38c4de09 Reviewed-on: https://boringssl-review.googlesource.com/7509 Reviewed-by: Mike Aizatsky <aizatsky@chromium.org> Reviewed-by: David Benjamin <davidben@google.com>
Diffstat (limited to 'FUZZING.md')
-rw-r--r--FUZZING.md15
1 files changed, 4 insertions, 11 deletions
diff --git a/FUZZING.md b/FUZZING.md
index 6416eeb..3474cbc 100644
--- a/FUZZING.md
+++ b/FUZZING.md
@@ -23,21 +23,14 @@ Then copy `libFuzzer.a` to the top-level of your BoringSSL source directory.
From the `build/` directory, you can then run the fuzzers. For example:
```
-./fuzz/cert -max_len=4000 -jobs=32 -workers=32 ../fuzz/cert_corpus/
+./fuzz/cert -max_len=3072 -jobs=32 -workers=32 ../fuzz/cert_corpus/
```
-The `max_len` argument is often important because, without it, libFuzzer defaults to limiting all test cases to 64 bytes, which is often insufficient for the formats that we wish to fuzz. The arguments to `jobs` and `workers` should be the number of cores that you wish to dedicate to fuzzing.
+The arguments to `jobs` and `workers` should be the number of cores that you wish to dedicate to fuzzing. By default, libFuzzer uses the largest test in the corpus (or 64 if empty) as the maximum test case length. The `max_len` argument overrides this.
-There are directories in `fuzz/` for each of the fuzzing tests which contain seed files for fuzzing. Some of the seed files were generated manually but many of them are “interesting” results generated by the fuzzing itself. (Where “interesting” means that it triggered a previously unknown path in the code.)
-
-Here are the recommended values of `max_len` for each test.
+The recommended values of `max_len` for each test may be found in `.options` files alongside the test source. These were determined by rounding up the length of the largest case in the corpus. When writing a new fuzzer, configure `max_len` in a similar file.
-| Test | `max_len` value |
-|-----------|-----------------|
-| `privkey` | 2048 |
-| `cert` | 3072 |
-| `server` | 1024 |
-| `client` | 4096 |
+There are directories in `fuzz/` for each of the fuzzing tests which contain seed files for fuzzing. Some of the seed files were generated manually but many of them are “interesting” results generated by the fuzzing itself. (Where “interesting” means that it triggered a previously unknown path in the code.)
## Minimising the corpuses