diff options
| author | Tomas Mraz <tomas@openssl.org> | 2023-11-03 08:52:43 +0100 |
|---|---|---|
| committer | Hugo Landau <hlandau@openssl.org> | 2023-11-06 08:50:08 +0100 |
| commit | 0e2e4b3e69d4012f47b6908c2d8a13ec4e2d40e0 (patch) | |
| tree | 5d815940b773f5f904d20fb01591111947543550 /ssl | |
| parent | Fix documentation regarding KMAC sizes (diff) | |
| download | openssl-0e2e4b3e69d4012f47b6908c2d8a13ec4e2d40e0.tar.xz openssl-0e2e4b3e69d4012f47b6908c2d8a13ec4e2d40e0.zip | |
ossl_quic_new(): Avoid dereferencing NULL qc during cleanup
Fixes Coverity 1548383
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)
Diffstat (limited to 'ssl')
| -rw-r--r-- | ssl/quic/quic_impl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ssl/quic/quic_impl.c b/ssl/quic/quic_impl.c index dd689865e4..399d1d2afd 100644 --- a/ssl/quic/quic_impl.c +++ b/ssl/quic/quic_impl.c @@ -382,7 +382,7 @@ SSL *ossl_quic_new(SSL_CTX *ctx) qc = OPENSSL_zalloc(sizeof(*qc)); if (qc == NULL) { QUIC_RAISE_NON_NORMAL_ERROR(NULL, ERR_R_CRYPTO_LIB, NULL); - goto err; + return NULL; } #if defined(OPENSSL_THREADS) if ((qc->mutex = ossl_crypto_mutex_new()) == NULL) { |