diff options
author | Nan Xiao <nan@chinadtrace.org> | 2021-04-01 13:55:04 +0800 |
---|---|---|
committer | Pauli <pauli@openssl.org> | 2021-04-07 09:08:19 +1000 |
commit | 493e78986f9677c2b321273da51c276b9a8182d8 (patch) | |
tree | 9e953f15778f32ed3d43e5dae08df5db47643a38 | |
parent | 0cfbc828e03ad69c50ae51e0c88920d90906498a (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.c | 4 |
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 |