From: Richard Levitte <levitte@openssl.org>
Date: Sun, 21 Jun 2015 17:19:17 +0000 (+0200)
Subject: Cleanup mttest.c : use BIO_free only, no preceding hacks
X-Git-Tag: OpenSSL_1_1_0-pre1~976
X-Git-Url: https://git.openssl.org/?p=openssl.git;a=commitdiff_plain;h=03b672deca60ab1fb272f1a1e206769da969bc9a;hp=964626957f79e07ed97756527cdc7e84007c60c9

Cleanup mttest.c : use BIO_free only, no preceding hacks

Since [sc]_ssl->[rw]bio aren't available, do not try to fiddle with
them.  Surely, a BIO_free on the "main" BIOs should be enough

Reviewed-by: Rich Salz <rsalz@openssl.org>
---

diff --git a/crypto/threads/mttest.c b/crypto/threads/mttest.c
index 7bb431778f..a82f86a6dd 100644
--- a/crypto/threads/mttest.c
+++ b/crypto/threads/mttest.c
@@ -594,6 +594,7 @@ int doit(char *ctx[4])
     SSL_set_shutdown(s_ssl, SSL_SENT_SHUTDOWN | SSL_RECEIVED_SHUTDOWN);
 
  err:
+#if 0
     /*
      * We have to set the BIO's to NULL otherwise they will be free()ed
      * twice.  Once when th s_ssl is SSL_free()ed and again when c_ssl is
@@ -615,6 +616,7 @@ int doit(char *ctx[4])
     /* The SSL's are optionally freed in the following calls */
     BIO_free(c_to_s);
     BIO_free(s_to_c);
+#endif
 
     BIO_free(c_bio);
     BIO_free(s_bio);