Browse Source

Use approx_time() instead of time(NULL) in some HS functions.

These were breaking our unittests.
George Kadianakis 6 years ago
parent
commit
5fb6f656df
2 changed files with 2 additions and 2 deletions
  1. 1 1
      src/or/hs_service.c
  2. 1 1
      src/or/voting_schedule.c

+ 1 - 1
src/or/hs_service.c

@@ -433,7 +433,7 @@ service_intro_point_new(const extend_info_t *ei, unsigned int is_legacy)
     if (BUG(intro_point_max_lifetime < intro_point_min_lifetime)) {
       goto err;
     }
-    ip->time_to_expire = time(NULL) +
+    ip->time_to_expire = approx_time() +
       crypto_rand_int_range(intro_point_min_lifetime,intro_point_max_lifetime);
   }
 

+ 1 - 1
src/or/voting_schedule.c

@@ -168,7 +168,7 @@ voting_schedule_get_next_valid_after_time(void)
 
  done:
   if (need_to_recalculate_voting_schedule) {
-    voting_schedule_recalculate_timing(get_options(), now);
+    voting_schedule_recalculate_timing(get_options(), approx_time());
     voting_schedule.created_on_demand = 1;
   }