aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Goulet <dgoulet@torproject.org>2017-10-26 09:52:15 -0400
committerDavid Goulet <dgoulet@torproject.org>2017-10-26 09:52:15 -0400
commitc4ef21ff5eff26182428d388e0614b4f0c5eed58 (patch)
tree0873acbc9ce8fbfcb75591bbd4d2c02b6e9d429a
parent4d2de821ce9ab7a79e1a8bf5123adafd4326d023 (diff)
downloadtor-c4ef21ff5eff26182428d388e0614b4f0c5eed58.tar.gz
tor-c4ef21ff5eff26182428d388e0614b4f0c5eed58.zip
fuzzing: Make hsdescv3 use the decoding API correctly
Fixes #21509 Signed-off-by: David Goulet <dgoulet@torproject.org>
-rw-r--r--changes/bug215093
-rw-r--r--src/test/fuzz/fuzz_hsdescv3.c4
2 files changed, 6 insertions, 1 deletions
diff --git a/changes/bug21509 b/changes/bug21509
new file mode 100644
index 0000000000..593a01ef20
--- /dev/null
+++ b/changes/bug21509
@@ -0,0 +1,3 @@
+ o Minor bugfixes (hidden service v3, fuzzing):
+ - Fix the hidden service v3 descriptor decoding fuzzing to use the latest
+ decoding API correctly. Fixes bug 21509; bugfix on 0.3.2.1-alpha.
diff --git a/src/test/fuzz/fuzz_hsdescv3.c b/src/test/fuzz/fuzz_hsdescv3.c
index 3fd21148f4..03c509e2e6 100644
--- a/src/test/fuzz/fuzz_hsdescv3.c
+++ b/src/test/fuzz/fuzz_hsdescv3.c
@@ -55,10 +55,12 @@ int
fuzz_main(const uint8_t *data, size_t sz)
{
hs_descriptor_t *desc = NULL;
+ uint8_t subcredential[DIGEST256_LEN];
char *fuzzing_data = tor_memdup_nulterm(data, sz);
+ memset(subcredential, 'A', sizeof(subcredential));
- hs_desc_decode_descriptor(fuzzing_data, NULL, &desc);
+ hs_desc_decode_descriptor(fuzzing_data, subcredential, &desc);
if (desc) {
log_debug(LD_GENERAL, "Decoding okay");
hs_descriptor_free(desc);