diff options
| author | Tomas Mraz <tomas@openssl.org> | 2023-11-03 08:57:21 +0100 |
|---|---|---|
| committer | Hugo Landau <hlandau@openssl.org> | 2023-11-06 07:50:57 +0000 |
| commit | 1ee0a9d8d3f351813e9db1d528c569c4c8ac3eac (patch) | |
| tree | b1ad565756aa0d355957ef95cbf17ffa5897b256 | |
| parent | e2c6a1d9a0df7a468f73a7ba62a33a860daa9bb6 (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)
(cherry picked from commit 3150dbe7cb71de1ee7040c6fdeb254c88e775b7c)
| -rw-r--r-- | test/quicapitest.c | 8 |
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; |