summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTomas Mraz <tomas@openssl.org>2023-11-03 08:57:21 +0100
committerHugo Landau <hlandau@openssl.org>2023-11-06 07:50:08 +0000
commit3150dbe7cb71de1ee7040c6fdeb254c88e775b7c (patch)
treead73eda2c408cd92efb26109a347eae94006053d
parent0e2e4b3e69d4012f47b6908c2d8a13ec4e2d40e0 (diff)
tparam_on_enc_ext(): Remove dead code in cleanup
Fixes Coverity 1548382 Reviewed-by: Hugo Landau <hlandau@openssl.org> Reviewed-by: Matt Caswell <matt@openssl.org> Reviewed-by: Paul Dale <pauli@openssl.org> (Merged from https://github.com/openssl/openssl/pull/22606)
-rw-r--r--test/quicapitest.c8
1 files changed, 2 insertions, 6 deletions
diff --git a/test/quicapitest.c b/test/quicapitest.c
index 77c9598c69..41cf0fc7a8 100644
--- a/test/quicapitest.c
+++ b/test/quicapitest.c
@@ -1994,12 +1994,8 @@ static int tparam_on_enc_ext(QTEST_FAULT *qtf, QTEST_ENCRYPTED_EXTENSIONS *ee,
rc = 1;
err:
- if (have_wpkt) {
- if (rc)
- WPACKET_finish(&wpkt);
- else
- WPACKET_cleanup(&wpkt);
- }
+ if (have_wpkt)
+ WPACKET_cleanup(&wpkt);
BUF_MEM_free(old_bufm);
BUF_MEM_free(new_bufm);
return rc;