From a348df6d8b049785ffaec4a56adda744a63a9581 Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Thu, 28 May 2015 12:41:00 -0400 Subject: [PATCH] Avoid dereferencing null on unit test failure for link handshakes. This fixes CID 1301368 -- found by coverity --- src/test/test_link_handshake.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/test/test_link_handshake.c b/src/test/test_link_handshake.c index bfdd6f3d1a..b3b65319aa 100644 --- a/src/test/test_link_handshake.c +++ b/src/test/test_link_handshake.c @@ -179,9 +179,11 @@ test_link_handshake_certs_ok(void *arg) tor_free(cell2); certs_cell_free(cc1); certs_cell_free(cc2); - circuitmux_free(chan1->base_.cmux); + if (chan1) + circuitmux_free(chan1->base_.cmux); tor_free(chan1); - circuitmux_free(chan2->base_.cmux); + if (chan2) + circuitmux_free(chan2->base_.cmux); tor_free(chan2); crypto_pk_free(key1); crypto_pk_free(key2);