From 1dbf6b8bedadd5d287ab44e1a387a89e2fc6c70b Mon Sep 17 00:00:00 2001 From: Frederic Lecaille Date: Thu, 12 Dec 2024 10:45:26 +0100 Subject: [PATCH] BUG/MINOR: quic: wrong logical statement in in_recovery_period() (BBR) A && logical operator was badly replaced by a || in this function which decides if BBR is in a recovery period. Must be backported to 3.1. --- src/quic_cc_bbr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/quic_cc_bbr.c b/src/quic_cc_bbr.c index 39fd673df..9a3b0a83e 100644 --- a/src/quic_cc_bbr.c +++ b/src/quic_cc_bbr.c @@ -1296,7 +1296,7 @@ static void bbr_update_control_parameters(struct bbr *bbr, static inline int in_recovery_period(struct quic_cc_path *p, uint32_t ts) { - return tick_isset(p->recovery_start_ts) || + return tick_isset(p->recovery_start_ts) && tick_is_le(ts, p->recovery_start_ts); }