summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Goulet <dgoulet@torproject.org>2018-10-30 11:36:36 -0400
committerDavid Goulet <dgoulet@torproject.org>2018-10-30 11:36:36 -0400
commit95559279e15393347c606eaaa323e0b6490782c2 (patch)
treefa1a3f974bf5890cb56af3141f1cf243a4bbde3d
parent6c9d678ff66d012b4c8009c6ad44409c7368ed69 (diff)
parent56f713b8a477b4203f3bfd8f3ad0952706dc9599 (diff)
downloadtor-95559279e15393347c606eaaa323e0b6490782c2.tar.gz
tor-95559279e15393347c606eaaa323e0b6490782c2.zip
Merge branch 'tor-github/pr/415' into maint-0.3.5
-rw-r--r--changes/ticket279954
-rw-r--r--src/feature/hs/hs_service.c10
2 files changed, 8 insertions, 6 deletions
diff --git a/changes/ticket27995 b/changes/ticket27995
new file mode 100644
index 0000000000..8c75425749
--- /dev/null
+++ b/changes/ticket27995
@@ -0,0 +1,4 @@
+ o Minor bugfixes (hidden service v3, client authorization):
+ - Fix an assert() when adding a client authorization for the first time
+ and then sending a HUP signal to the service. Before that, tor would
+ stop abruptly. Fixes bug 27995; bugfix on 0.3.5.1-alpha.
diff --git a/src/feature/hs/hs_service.c b/src/feature/hs/hs_service.c
index 78654bfb23..aec2aa4381 100644
--- a/src/feature/hs/hs_service.c
+++ b/src/feature/hs/hs_service.c
@@ -1924,12 +1924,10 @@ build_service_desc_keys(const hs_service_t *service,
goto end;
}
- /* Random a descriptor cookie to be used as a part of a key to encrypt the
- * descriptor, if the client auth is enabled. */
- if (service->config.is_client_auth_enabled) {
- crypto_strongest_rand(desc->descriptor_cookie,
- sizeof(desc->descriptor_cookie));
- }
+ /* Random descriptor cookie to be used as a part of a key to encrypt the
+ * descriptor, only if the client auth is enabled will it be used. */
+ crypto_strongest_rand(desc->descriptor_cookie,
+ sizeof(desc->descriptor_cookie));
/* Success. */
ret = 0;