diff options
author | Matt Caswell <matt@openssl.org> | 2023-10-27 16:11:57 +0100 |
---|---|---|
committer | Hugo Landau <hlandau@openssl.org> | 2023-10-30 08:01:58 +0000 |
commit | 10dfd796c9c25dd78aa88cf84629a0418b8f0866 (patch) | |
tree | 0ded0545ccd37f9117b6281ca735ec4cbe1fd3b5 /ssl | |
parent | 3a29ac268470d5a1b8f34bfc5dae01308eedc6ca (diff) |
If the loss detection timer has fired we may not have lost packets
We calculate the delay from the point that a packet arrives until it will
be counted as lost based on rtt info. Looking at all the packets we can
then calculate the earliest time that a packet will be counted as lost.
When that timer fires the latest rtt info may have changed and therefore
the packet may no longer be counted as lost yet.
We should not assume that just because the ackm timeout has fired that
there will definitely be lost packets.
Fixes #22538
Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/22541)
Diffstat (limited to 'ssl')
-rw-r--r-- | ssl/quic/quic_ackm.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ssl/quic/quic_ackm.c b/ssl/quic/quic_ackm.c index 728a186d3b..75a1e5741a 100644 --- a/ssl/quic/quic_ackm.c +++ b/ssl/quic/quic_ackm.c @@ -1301,8 +1301,8 @@ int ossl_ackm_on_timeout(OSSL_ACKM *ackm) if (!ossl_time_is_zero(earliest_loss_time)) { /* Time threshold loss detection. */ lost_pkts = ackm_detect_and_remove_lost_pkts(ackm, pkt_space); - assert(lost_pkts != NULL); - ackm_on_pkts_lost(ackm, pkt_space, lost_pkts, /*pseudo=*/0); + if (lost_pkts != NULL) + ackm_on_pkts_lost(ackm, pkt_space, lost_pkts, /*pseudo=*/0); ackm_set_loss_detection_timer(ackm); return 1; } |