summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNan Xiao <nan@chinadtrace.org>2021-04-01 13:55:04 +0800
committerPauli <pauli@openssl.org>2021-04-07 09:08:19 +1000
commit493e78986f9677c2b321273da51c276b9a8182d8 (patch)
tree9e953f15778f32ed3d43e5dae08df5db47643a38
parent0cfbc828e03ad69c50ae51e0c88920d90906498a (diff)
Fix potential double free in sslapitest.c
Reviewed-by: Tomas Mraz <tomas@openssl.org> Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from https://github.com/openssl/openssl/pull/14758)
-rw-r--r--test/sslapitest.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/test/sslapitest.c b/test/sslapitest.c
index 3e5d532bf4..31b36b23b1 100644
--- a/test/sslapitest.c
+++ b/test/sslapitest.c
@@ -2743,8 +2743,10 @@ static int execute_test_ssl_bio(int pop_ssl, bio_change_t change_bio)
/* Verify changing the rbio/wbio directly does not cause leaks */
if (change_bio != NO_BIO_CHANGE) {
- if (!TEST_ptr(membio2 = BIO_new(BIO_s_mem())))
+ if (!TEST_ptr(membio2 = BIO_new(BIO_s_mem()))) {
+ ssl = NULL;
goto end;
+ }
if (change_bio == CHANGE_RBIO)
SSL_set0_rbio(ssl, membio2);
else