Browse Source

Fix a NULL dereference on unit test failure. CID 1353177.

Nick Mathewson 8 years ago
parent
commit
31c96a3699
1 changed files with 4 additions and 2 deletions
  1. 4 2
      src/test/test_routerlist.c

+ 4 - 2
src/test/test_routerlist.c

@@ -434,8 +434,10 @@ test_routerlist_router_is_already_dir_fetching(void *arg)
 
  done:
   /* If a connection is never set up, connection_free chokes on it. */
-  buf_free(mocked_connection->inbuf);
-  buf_free(mocked_connection->outbuf);
+  if (mocked_connection) {
+    buf_free(mocked_connection->inbuf);
+    buf_free(mocked_connection->outbuf);
+  }
   tor_free(mocked_connection);
   UNMOCK(connection_get_by_type_addr_port_purpose);
 }