Browse Source

Reinstate a couple of teardown_capture_of_logs that I missed

Patch from rubiate. See #19999
Nick Mathewson 7 years ago
parent
commit
63e34e9e49
2 changed files with 2 additions and 7 deletions
  1. 1 4
      src/test/test_tortls.c
  2. 1 3
      src/test/test_util.c

+ 1 - 4
src/test/test_tortls.c

@@ -2278,7 +2278,6 @@ test_tortls_finish_handshake(void *ignored)
 
   X509 *c1 = read_cert_from(validCertString);
   SESS_CERT_local *sess = NULL;
-  int log_level = 0;
 
   ctx = SSL_CTX_new(method);
 
@@ -2298,7 +2297,6 @@ test_tortls_finish_handshake(void *ignored)
   expect_single_log_msg_containing("For some reason, wasV2Handshake didn't "
                                    "get set.");
   teardown_capture_of_logs();
-  log_level = 0;
 
   tls->wasV2Handshake = 1;
   ret = tor_tls_finish_handshake(tls);
@@ -2337,8 +2335,7 @@ test_tortls_finish_handshake(void *ignored)
   tor_free(tls);
   SSL_CTX_free(ctx);
   tor_free(method);
-  if (log_level)
-    teardown_capture_of_logs();
+  teardown_capture_of_logs();
 }
 #endif
 

+ 1 - 3
src/test/test_util.c

@@ -262,7 +262,6 @@ test_util_time(void *arg)
   time_t t_res;
   int i;
   struct timeval tv;
-  int old_log_level = 0;
 
   /* Test tv_udiff and tv_mdiff */
 
@@ -1112,8 +1111,7 @@ test_util_time(void *arg)
 #undef CHECK_TIMEGM_ARG_OUT_OF_RANGE
 
  done:
-  if (old_log_level)
-    teardown_capture_of_logs();
+  teardown_capture_of_logs();
 }
 
 static void