ns: Move dirvote_get_voter_sig_by_alg() to networkstatus.c

It makes more sense to be in networkstatus.c so move it there and rename it
with the "networkstatus_" prefix.

Part of #25988

Signed-off-by: David Goulet <dgoulet@torproject.org>

diff --git a/src/or/dirauth/dirvote.c b/src/or/dirauth/dirvote.c
index 36f328d6c57a4a48a8c94f527d8027ef801bdbb2..f11b871e50d02b72bd1b58c57097a7fa621c0cab 100644 (file)
--- a/src/or/dirauth/dirvote.c
+++ b/src/or/dirauth/dirvote.c
@@ -2516,7 +2516,7 @@ networkstatus_add_detached_signatures(networkstatus_t *target,
       continue;
     }
 
-    old_sig = dirvote_get_voter_sig_by_alg(target_voter, sig->alg);
+    old_sig = networkstatus_get_voter_sig_by_alg(target_voter, sig->alg);
 
     /* If the target already has a good signature from this voter, then skip
      * this one. */

diff --git a/src/or/dirvote_common.c b/src/or/dirvote_common.c
index a28a441d6447558700035da0f19d1997de51bc3b..354ee0ce72fcc7198569a0172ac7bc0b99246ffb 100644 (file)
--- a/src/or/dirvote_common.c
+++ b/src/or/dirvote_common.c
@@ -179,17 +179,3 @@ dirvote_recalculate_timing(const or_options_t *options, time_t now)
   voting_schedule_free(new_voting_schedule);
 }
 
-/** Return the signature made by <b>voter</b> using the algorithm
- * <b>alg</b>, or NULL if none is found. */
-document_signature_t *
-dirvote_get_voter_sig_by_alg(const networkstatus_voter_info_t *voter,
-                             digest_algorithm_t alg)
-{
-  if (!voter->sigs)
-    return NULL;
-  SMARTLIST_FOREACH(voter->sigs, document_signature_t *, sig,
-                    if (sig->alg == alg)
-                    return sig);
-  return NULL;
-}
-

diff --git a/src/or/dirvote_common.h b/src/or/dirvote_common.h
index 857b8497ee7f8c903879cda064c033e1d2297266..096e5b030002a2f42f7dc42b4927ee3683dde376 100644 (file)
--- a/src/or/dirvote_common.h
+++ b/src/or/dirvote_common.h
@@ -58,9 +58,5 @@ time_t dirvote_get_start_of_next_interval(time_t now,
                                           int offset);
 time_t dirvote_get_next_valid_after_time(void);
 
-document_signature_t *dirvote_get_voter_sig_by_alg(
-                           const networkstatus_voter_info_t *voter,
-                           digest_algorithm_t alg);
-
 #endif /* TOR_DIRVOTE_COMMON_H */
 

diff --git a/src/or/networkstatus.c b/src/or/networkstatus.c
index 1267d9d6bcf475283b7efeda01deb29ae19810a2..99f1490203a77a9b0ba4076060f733987fb602a3 100644 (file)
--- a/src/or/networkstatus.c
+++ b/src/or/networkstatus.c
@@ -390,6 +390,20 @@ networkstatus_get_voter_by_id(networkstatus_t *vote,
   return NULL;
 }
 
+/** Return the signature made by <b>voter</b> using the algorithm
+ * <b>alg</b>, or NULL if none is found. */
+document_signature_t *
+networkstatus_get_voter_sig_by_alg(const networkstatus_voter_info_t *voter,
+                                   digest_algorithm_t alg)
+{
+  if (!voter->sigs)
+    return NULL;
+  SMARTLIST_FOREACH(voter->sigs, document_signature_t *, sig,
+                    if (sig->alg == alg)
+                    return sig);
+  return NULL;
+}
+
 /** Check whether the signature <b>sig</b> is correctly signed with the
  * signing key in <b>cert</b>.  Return -1 if <b>cert</b> doesn't match the
  * signing key; otherwise set the good_signature or bad_signature flag on

diff --git a/src/or/networkstatus.h b/src/or/networkstatus.h
index bd782920508a72e6f945a9ef91c9874b20998626..0c325959d78a4f4def2c58c59f9f9b4ce9d9bab9 100644 (file)
--- a/src/or/networkstatus.h
+++ b/src/or/networkstatus.h
@@ -30,6 +30,10 @@ void ns_detached_signatures_free_(ns_detached_signatures_t *s);
 networkstatus_voter_info_t *networkstatus_get_voter_by_id(
                                        networkstatus_t *vote,
                                        const char *identity);
+document_signature_t *networkstatus_get_voter_sig_by_alg(
+                                    const networkstatus_voter_info_t *voter,
+                                    digest_algorithm_t alg);
+
 int networkstatus_check_consensus_signature(networkstatus_t *consensus,
                                             int warn);
 int networkstatus_check_document_signature(const networkstatus_t *consensus,

diff --git a/src/or/routerparse.c b/src/or/routerparse.c
index 3a55c00cc1e00c881c91ca4e46cf512c49d10e62..e4833b441950964a312af9329442936b155ddc60 100644 (file)
--- a/src/or/routerparse.c
+++ b/src/or/routerparse.c
@@ -3912,7 +3912,7 @@ networkstatus_parse_vote_from_string(const char *s, const char **eos_out,
       }
     }
 
-    if (dirvote_get_voter_sig_by_alg(v, sig->alg)) {
+    if (networkstatus_get_voter_sig_by_alg(v, sig->alg)) {
       /* We already parsed a vote with this algorithm from this voter. Use the
          first one. */
       log_fn(LOG_PROTOCOL_WARN, LD_DIR, "We received a networkstatus "