prop224: Pick the right hsdir index based on descriptor.

There was a bug in upload_descriptor_to_all() where we picked between
first and second hsdir index based on which time segment we are. That's
not right and instead we should be uploading our two descriptors using a
different hsdir index every time. That is, upload first descriptor using
first hsdir index, and upload second descriptor using second hdsir index.

Also simplify stuff in pick_hdsir_v3() since that's only used to fetch
descriptors and hence we can just always use the fetch hsdir index.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/or/hs_common.h b/src/or/hs_common.h
index 77b4e6544b..5851578fd6 100644
--- a/src/or/hs_common.h
+++ b/src/or/hs_common.h
@@ -146,8 +146,8 @@ typedef struct hsdir_index_t {
   uint8_t fetch[DIGEST256_LEN];
 
   /* HSDir index used by services to store their first and second
-   * descriptor. The first descriptor is the one that uses older TP and SRV
-   * values than the second one. */
+   * descriptor. The first descriptor is chronologically older than the second
+   * one and uses older TP and SRV values. */
   uint8_t store_first[DIGEST256_LEN];
   uint8_t store_second[DIGEST256_LEN];
 } hsdir_index_t;