sr: rename srv_dup() to sr_srv_dup()

diff --git a/src/or/shared_random.c b/src/or/shared_random.c
index c3e6409771258147a4fc390aadc71b5f666bd520..ddb6de1a5933f2592b31a76f8a0a22cf0882e808 100644 (file)
--- a/src/or/shared_random.c
+++ b/src/or/shared_random.c
@@ -113,7 +113,7 @@ static int32_t num_srv_agreements_from_vote;
 
 /* Return a heap allocated copy of the SRV <b>orig</b>. */
 sr_srv_t *
-srv_dup(const sr_srv_t *orig)
+sr_srv_dup(const sr_srv_t *orig)
 {
   sr_srv_t *duplicate = NULL;
 
@@ -1318,8 +1318,8 @@ sr_act_post_consensus(const networkstatus_t *consensus)
      * decided by the majority. */
     sr_state_unset_fresh_srv();
     /* Set the SR values from the given consensus. */
-    sr_state_set_previous_srv(srv_dup(consensus->sr_info.previous_srv));
-    sr_state_set_current_srv(srv_dup(consensus->sr_info.current_srv));
+    sr_state_set_previous_srv(sr_srv_dup(consensus->sr_info.previous_srv));
+    sr_state_set_current_srv(sr_srv_dup(consensus->sr_info.current_srv));
   }
 
   /* Prepare our state so that it's ready for the next voting period. */

diff --git a/src/or/shared_random.h b/src/or/shared_random.h
index 68a1019794ab7afef126cd0f5424397e4cb534b4..c640470c16be9791e82073e45ee768a5fc9a9e0d 100644 (file)
--- a/src/or/shared_random.h
+++ b/src/or/shared_random.h
@@ -129,7 +129,7 @@ const char *sr_commit_get_rsa_fpr(const sr_commit_t *commit)
 void sr_compute_srv(void);
 sr_commit_t *sr_generate_our_commit(time_t timestamp,
                                     const authority_cert_t *my_rsa_cert);
-sr_srv_t *srv_dup(const sr_srv_t *orig);
+sr_srv_t *sr_srv_dup(const sr_srv_t *orig);
 
 #ifdef SHARED_RANDOM_PRIVATE
 

diff --git a/src/or/shared_random_state.c b/src/or/shared_random_state.c
index e99817106cba59689aa9757848e92d725fbada23..50c6d3dd7a7d60031a78c269f6aea1de5bd50ec7 100644 (file)
--- a/src/or/shared_random_state.c
+++ b/src/or/shared_random_state.c
@@ -1037,7 +1037,7 @@ state_rotate_srv(void)
   /* First delete previous SRV from the state. Object will be freed. */
   state_del_previous_srv();
   /* Set previous SRV to a copy of the current one. */
-  sr_state_set_previous_srv(srv_dup(sr_state_get_current_srv()));
+  sr_state_set_previous_srv(sr_srv_dup(sr_state_get_current_srv()));
   /* Free and NULL the current srv. */
   sr_state_set_current_srv(NULL);
 }

diff --git a/src/test/test_shared_random.c b/src/test/test_shared_random.c
index 5d3e574fac4c13df0178d4b26d28c482ca7b8223..93125e5135f09e8d6fa70cecadf7d12bf340ab23 100644 (file)
--- a/src/test/test_shared_random.c
+++ b/src/test/test_shared_random.c
@@ -932,7 +932,7 @@ test_utils(void *arg)
 {
   (void) arg;
 
-  /* Testing srv_dup(). */
+  /* Testing sr_srv_dup(). */
   {
     sr_srv_t *srv = NULL, *dup_srv = NULL;
     const char *srv_value =
@@ -940,7 +940,7 @@ test_utils(void *arg)
     srv = tor_malloc_zero(sizeof(*srv));
     srv->num_reveals = 42;
     memcpy(srv->value, srv_value, sizeof(srv->value));
-    dup_srv = srv_dup(srv);
+    dup_srv = sr_srv_dup(srv);
     tt_assert(dup_srv);
     tt_u64_op(dup_srv->num_reveals, ==, srv->num_reveals);
     tt_mem_op(dup_srv->value, OP_EQ, srv->value, sizeof(srv->value));
@@ -1055,7 +1055,7 @@ test_state_transition(void *arg)
     test_sr_setup_srv(1);
     tt_assert(sr_state_get_current_srv());
     /* Take a copy of the data, because the state owns the pointer */
-    cur = srv_dup(sr_state_get_current_srv());
+    cur = sr_srv_dup(sr_state_get_current_srv());
     tt_assert(cur);
     /* After, the previous SRV should be the same as the old current SRV, and
      * the current SRV should be set to NULL */
@@ -1073,7 +1073,7 @@ test_state_transition(void *arg)
     test_sr_setup_srv(1);
     tt_assert(sr_state_get_current_srv());
     /* Take a copy of the data, because the state owns the pointer */
-    cur = srv_dup(sr_state_get_current_srv());
+    cur = sr_srv_dup(sr_state_get_current_srv());
     set_sr_phase(SR_PHASE_REVEAL);
     MOCK(get_my_v3_authority_cert, get_my_v3_authority_cert_m);
     new_protocol_run(now);