diff options
| author | Todd Short <tshort@akamai.com> | 2022-10-19 10:13:35 -0400 |
|---|---|---|
| committer | Pauli <pauli@openssl.org> | 2022-10-21 10:33:40 +1100 |
| commit | d06d5d6b68f39c7f75f1130f984efa78c291fb57 (patch) | |
| tree | 0976e0b458ad90e6c83fbc136c3f4abfb84ed90e /ssl/statem | |
| parent | 5e569f0a2e11a59cab7b6f525865232e7770e2f0 (diff) | |
Fix coverity 1516094 uninit
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19440)
Diffstat (limited to 'ssl/statem')
| -rw-r--r-- | ssl/statem/statem_clnt.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/ssl/statem/statem_clnt.c b/ssl/statem/statem_clnt.c index ac4fb4e00d..3021bc1952 100644 --- a/ssl/statem/statem_clnt.c +++ b/ssl/statem/statem_clnt.c @@ -3732,7 +3732,10 @@ CON_FUNC_RETURN tls_construct_client_compressed_certificate(SSL_CONNECTION *sc, err: SSLfatal(sc, SSL_AD_INTERNAL_ERROR, ERR_R_INTERNAL_ERROR); out: - WPACKET_cleanup(&tmppkt); + if (buf != NULL) { + /* If |buf| is NULL, then |tmppkt| could not have been initialized */ + WPACKET_cleanup(&tmppkt); + } BUF_MEM_free(buf); COMP_CTX_free(comp); return ret; |