Skip to content
Snippets Groups Projects
Commit d583459d authored by Nick Mathewson's avatar Nick Mathewson :game_die:
Browse files

Fix some memory errors in the recent coverity fixes.

Found by asan on travis :/
parent ea14d340
Branches
Tags
No related merge requests found
......@@ -677,6 +677,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);
......
......@@ -207,6 +207,7 @@ test_link_specifier(void *arg)
tt_int_op(link_specifier_get_un_ipv4_port(ls), ==, spec.u.ap.port);
link_specifier_free(ls);
ls = NULL;
tor_free(b64);
}
......@@ -236,6 +237,7 @@ test_link_specifier(void *arg)
tt_int_op(link_specifier_get_un_ipv6_port(ls), ==, spec.u.ap.port);
link_specifier_free(ls);
ls = NULL;
tor_free(b64);
}
......@@ -263,6 +265,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);
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment