test_tlsext_status_type(): Avoid leaking of previously allocated data

Fixes Coverity 1659226, 1659224, 1659223

Reviewed-by: David von Oheimb <david.von.oheimb@siemens.com>
Reviewed-by: Viktor Dukhovni <viktor@openssl.org>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/28105)

diff --git a/test/sslapitest.c b/test/sslapitest.c
index 216bb1b9ed0b780aab9bc9c9f8114340ec15cc6a..d4450b01d49f1a2ecd685d3c2dca9dffd13c514f 100644 (file)
--- a/test/sslapitest.c
+++ b/test/sslapitest.c
@@ -1984,7 +1984,7 @@ static int test_tlsext_status_type(void)
     if (!create_ssl_ctx_pair(libctx, TLS_server_method(), TLS_client_method(),
                              TLS1_VERSION, 0,
                              &sctx, &cctx, leaf, skey))
-        return 0;
+        goto end;
     if (SSL_CTX_use_certificate_chain_file(sctx, leaf_chain) <= 0)
         goto end;
     if (SSL_CTX_get_tlsext_status_type(cctx) != -1)