]> git.ipfire.org Git - thirdparty/tor.git/commitdiff
asan does not like TO_CONN(NULL)
authorNick Mathewson <nickm@torproject.org>
Mon, 22 Feb 2016 20:45:37 +0000 (15:45 -0500)
committerNick Mathewson <nickm@torproject.org>
Mon, 22 Feb 2016 20:45:37 +0000 (15:45 -0500)
changes/test_asan [new file with mode: 0644]
src/test/test_dir_handle_get.c

diff --git a/changes/test_asan b/changes/test_asan
new file mode 100644 (file)
index 0000000..ce0aa97
--- /dev/null
@@ -0,0 +1,3 @@
+  o Testing:
+    - Fix several warnings from clang's address sanitizer produced in the
+      unit tests.
index e1ac50e66eaae93c68c6b2593646b6e7dda14d3f..c81ed7ba89654ea2c1bcfbe73ba477b4078bafdd 100644 (file)
@@ -2255,7 +2255,6 @@ status_vote_next_consensus_signatures_test(char **header, char **body,
 static void
 test_dir_handle_get_status_vote_next_consensus_signatures_not_found(void* data)
 {
-  dir_connection_t *conn = NULL;
   char *header = NULL, *body = NULL;
   size_t body_used;
   (void) data;
@@ -2266,7 +2265,6 @@ test_dir_handle_get_status_vote_next_consensus_signatures_not_found(void* data)
   tt_str_op(NOT_FOUND, OP_EQ, header);
 
   done:
-    connection_free_(TO_CONN(conn));
     tor_free(header);
     tor_free(body);
 }
@@ -2308,7 +2306,6 @@ test_dir_handle_get_status_vote_next_consensus_signatures(void* data)
 static void
 test_dir_handle_get_status_vote_next_consensus_signatures_busy(void* data)
 {
-  dir_connection_t *conn = NULL;
   char *header = NULL, *body = NULL;
   size_t body_used;
   (void) data;
@@ -2328,7 +2325,6 @@ test_dir_handle_get_status_vote_next_consensus_signatures_busy(void* data)
   done:
     UNMOCK(get_options);
     NS_UNMOCK(dirvote_get_pending_detached_signatures);
-    connection_free_(TO_CONN(conn));
     tor_free(header);
     tor_free(body);
     or_options_free(mock_options); mock_options = NULL;