diff options
| author | George Kadianakis <desnacked@riseup.net> | 2017-09-26 15:23:23 +0300 |
|---|---|---|
| committer | David Goulet <dgoulet@torproject.org> | 2017-09-27 09:24:35 -0400 |
| commit | 178a2bbcff74b7f9df2a192f45f1f91f3a9f935d (patch) | |
| tree | 24dcdcbc91d631b5899978b58f1e84a25aa5d1b9 | |
| parent | 6051375ccac45f68df48e6ce008f6849d160ce63 (diff) | |
| download | tor-178a2bbcff74b7f9df2a192f45f1f91f3a9f935d.tar.gz tor-178a2bbcff74b7f9df2a192f45f1f91f3a9f935d.zip | |
prop224: Fix one last instance of get_time_period_num() with now.
When purging last HSDir requests, we used time(NULL) for computing the
service blinded key but in all other places in our codebase we actually
use the consensus times. That can cause wrong behavior if the consensus
is in a different time period than time(NULL).
This commit is required for proper purging of HSDir requests.
| -rw-r--r-- | src/or/hs_client.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/or/hs_client.c b/src/or/hs_client.c index 201e808180..560ccf8499 100644 --- a/src/or/hs_client.c +++ b/src/or/hs_client.c @@ -100,7 +100,7 @@ purge_hid_serv_request(const ed25519_public_key_t *identity_pk) * from the previous time period. That is fine because they will expire at * some point and we don't care about those anymore. */ hs_build_blinded_pubkey(identity_pk, NULL, 0, - hs_get_time_period_num(approx_time()), &blinded_pk); + hs_get_time_period_num(0), &blinded_pk); if (BUG(ed25519_public_to_base64(base64_blinded_pk, &blinded_pk) < 0)) { return; } |