diff options
| author | George Kadianakis <desnacked@riseup.net> | 2017-08-04 12:21:14 +0300 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2017-08-08 20:29:34 -0400 |
| commit | b89d2fa1db2379bffd2e2b4c851c3facc57b6ed8 (patch) | |
| tree | 6f4769adc1ddfd5d4c82958eba03ce5575bd113c /src/or/shared_random.h | |
| parent | 440eaa9b22573cdb0d38bf5c13200cc1077a453f (diff) | |
| download | tor-b89d2fa1db2379bffd2e2b4c851c3facc57b6ed8.tar.gz tor-b89d2fa1db2379bffd2e2b4c851c3facc57b6ed8.zip | |
Don't set HSDir index if we don't have a live consensus.
We also had to alter the SRV functions to take a consensus as optional
input, since we might be setting our HSDir index using a consensus that
is currently being processed and won't be returned by the
networkstatus_get_live_consensus() function.
This change has two results:
a) It makes sure we are using a fresh consensus with the right SRV value
when we are calculating the HSDir hash ring.
b) It ensures that we will not use the sr_get_current/previous()
functions when we don't have a consensus which would have falsely
triggered the disaster SRV logic.
Diffstat (limited to 'src/or/shared_random.h')
| -rw-r--r-- | src/or/shared_random.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/or/shared_random.h b/src/or/shared_random.h index 58ea360df0..76d5b95422 100644 --- a/src/or/shared_random.h +++ b/src/or/shared_random.h @@ -130,8 +130,8 @@ sr_commit_t *sr_generate_our_commit(time_t timestamp, char *sr_get_current_for_control(void); char *sr_get_previous_for_control(void); -const sr_srv_t *sr_get_current(void); -const sr_srv_t *sr_get_previous(void); +const sr_srv_t *sr_get_current(const networkstatus_t *ns); +const sr_srv_t *sr_get_previous(const networkstatus_t *ns); #ifdef SHARED_RANDOM_PRIVATE |