BUILD: quic: Move an ASSUME_NONNULL() for variable which is not null
authorFrederic Lecaille <flecaille@haproxy.com>
Tue, 21 Jan 2025 15:26:42 +0000 (16:26 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Thu, 23 Jan 2025 10:22:39 +0000 (11:22 +0100)
commit0ef4f1bdbd7de903fc770b33da20eb741a5ac0a4
tree549c8e9cd550a59f572de46f72310ef659f88fbb
parent1264b812bc10699d912d3bedcc73b0a1ac059cf0
BUILD: quic: Move an ASSUME_NONNULL() for variable which is not null

Some new compilers warn that <oldest_lost> variable can be null even this cannot be
the case as mentioned by the comment about an already present ASSUME_NONNULL()
call comment as follows:

src/quic_loss.c: In function ‘qc_release_lost_pkts’:
src/quic_loss.c:307:86: error: potential null pointer dereference [-Werror=null-dereference]
  307 |   unsigned int period = newest_lost->time_sent_ms - oldest_lost->time_sent_ms;
      |                                                     ~~~~~~~~~~~^~~~~~~~~~~~~~

Move up this ASSUME_NONNULL() statement to please these compiler.

Must be backported as far as 2.6 to easy any further backport around this code part.

(cherry picked from commit 1f099db7e2ca978b467f0d524261af1d588d1d0a)
[cf: ALREADY_CHECKED() is moved because ASSUME_NONNULL() does not exist]
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
src/quic_loss.c