tor

commit ed84dab8b4cd691dcdc0fecfb9c451bb0ffd930a
parent 6e9d5fc4ec40136d173be680a85a9bfb9b99fea4
Author: Nick Mathewson <nickm@torproject.org>
Date:   Fri, 29 Jun 2018 13:49:43 -0400

Merge branch 'maint-0.3.1' into maint-0.3.2

Diffstat:
M
src/test/test_channelpadding.c
 | 
1
+
M
src/test/test_hs_descriptor.c
 | 
3
+++
M
src/test/test_shared_random.c
 | 
1
+

3 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/src/test/test_channelpadding.c b/src/test/test_channelpadding.c
@@ -680,6 +680,7 @@ test_channelpadding_consensus(void *arg)
   memcpy(relay->identity_digest,
           ((channel_tls_t *)chan)->conn->identity_digest, DIGEST_LEN);
   smartlist_add(current_md_consensus->routerstatus_list, relay);
+  relay = NULL; /* Prevent double-free */
 
   tried_to_write_cell = 0;
   decision = channelpadding_decide_to_pad_channel(chan);
diff --git a/src/test/test_hs_descriptor.c b/src/test/test_hs_descriptor.c
@@ -208,6 +208,7 @@ test_link_specifier(void *arg)
     tt_int_op(link_specifier_get_un_ipv4_port(ls), OP_EQ, spec.u.ap.port);
 
     link_specifier_free(ls);
+    ls = NULL;
     tor_free(b64);
   }
 
@@ -238,6 +239,7 @@ test_link_specifier(void *arg)
     tt_int_op(link_specifier_get_un_ipv6_port(ls), OP_EQ, spec.u.ap.port);
 
     link_specifier_free(ls);
+    ls = NULL;
     tor_free(b64);
   }
 
@@ -265,6 +267,7 @@ test_link_specifier(void *arg)
     tt_mem_op(spec.u.legacy_id, OP_EQ, id, DIGEST_LEN);
 
     link_specifier_free(ls);
+    ls = NULL;
     tor_free(b64);
   }
 
diff --git a/src/test/test_shared_random.c b/src/test/test_shared_random.c
@@ -869,6 +869,7 @@ test_sr_setup_commits(void)
   /* Now during REVEAL phase save commit D by restoring its reveal. */
   set_sr_phase(SR_PHASE_REVEAL);
   save_commit_to_state(place_holder);
+  place_holder = NULL;
   tt_str_op(commit_d->encoded_reveal, OP_EQ,
             "DDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDDD");
   /* Go back to an empty encoded reveal value. */