mirror of https://github.com/openssl/openssl.git
QUIC APL: Fix QUIC_TLS free order
Reviewed-by: Matt Caswell <matt@openssl.org> Reviewed-by: Neil Horman <nhorman@openssl.org> (Merged from https://github.com/openssl/openssl/pull/23334)
This commit is contained in:
parent
d2ee7ceff0
commit
882c77e93e
|
|
@ -524,6 +524,9 @@ err:
|
||||||
QUIC_NEEDS_LOCK
|
QUIC_NEEDS_LOCK
|
||||||
static void qc_cleanup(QUIC_CONNECTION *qc, int have_lock)
|
static void qc_cleanup(QUIC_CONNECTION *qc, int have_lock)
|
||||||
{
|
{
|
||||||
|
SSL_free(qc->tls);
|
||||||
|
qc->tls = NULL;
|
||||||
|
|
||||||
ossl_quic_channel_free(qc->ch);
|
ossl_quic_channel_free(qc->ch);
|
||||||
qc->ch = NULL;
|
qc->ch = NULL;
|
||||||
|
|
||||||
|
|
@ -543,9 +546,6 @@ static void qc_cleanup(QUIC_CONNECTION *qc, int have_lock)
|
||||||
ossl_quic_engine_free(qc->engine);
|
ossl_quic_engine_free(qc->engine);
|
||||||
qc->engine = NULL;
|
qc->engine = NULL;
|
||||||
|
|
||||||
SSL_free(qc->tls);
|
|
||||||
qc->tls = NULL;
|
|
||||||
|
|
||||||
if (have_lock)
|
if (have_lock)
|
||||||
/* tsan doesn't like freeing locked mutexes */
|
/* tsan doesn't like freeing locked mutexes */
|
||||||
ossl_crypto_mutex_unlock(qc->mutex);
|
ossl_crypto_mutex_unlock(qc->mutex);
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue