]> git.ipfire.org Git - thirdparty/tor.git/commitdiff
prop224: Fix one last instance of get_time_period_num() with now.
authorGeorge Kadianakis <desnacked@riseup.net>
Tue, 26 Sep 2017 12:23:23 +0000 (15:23 +0300)
committerDavid Goulet <dgoulet@torproject.org>
Wed, 27 Sep 2017 13:24:35 +0000 (09:24 -0400)
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.

src/or/hs_client.c

index 201e808180ee32f4d0db56e6bbdd50cbe65e9514..560ccf849981df2df91c4af411acb9dbd53971ed 100644 (file)
@@ -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;
   }