diff options
author | Hugo Landau <hlandau@openssl.org> | 2023-05-02 08:29:43 +0100 |
---|---|---|
committer | Hugo Landau <hlandau@openssl.org> | 2023-05-12 14:47:14 +0100 |
commit | 3a61a96c1e43694199f5ec0887d0ad21a4650e3f (patch) | |
tree | 1c953bce423941c66b80e0597adceba8509575e2 /ssl/quic | |
parent | 47238a3d3f87c8381f5c9e013e6fc7c4e0d34557 (diff) |
QUIC MSST: Clarify default XSO transitions
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/20765)
Diffstat (limited to 'ssl/quic')
-rw-r--r-- | ssl/quic/quic_impl.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/ssl/quic/quic_impl.c b/ssl/quic/quic_impl.c index 18a647bfa5..0826a170d4 100644 --- a/ssl/quic/quic_impl.c +++ b/ssl/quic/quic_impl.c @@ -520,7 +520,13 @@ static void qc_set_default_xso_keep_ref(QUIC_CONNECTION *qc, QUIC_XSO *xso, * Changing from not having a default XSO to having one. The new XSO * will have had a reference to the QC we need to drop to avoid a * circular reference. + * + * Currently we never change directly from one default XSO to + * another, though this function would also still be correct if this + * weren't the case. */ + assert(*old_xso == NULL); + CRYPTO_DOWN_REF(&qc->ssl.references, &refs, &qc->ssl.lock); assert(refs > 0); } |