]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: quic: Possible infinite loop in quic_build_post_handshake_frames()
authorFrédéric Lécaille <flecaille@haproxy.com>
Thu, 11 Aug 2022 10:04:07 +0000 (12:04 +0200)
committerWilly Tarreau <w@1wt.eu>
Thu, 11 Aug 2022 12:33:43 +0000 (14:33 +0200)
commit96d08d37d97c2f250c4436b920ae708d45135708
treed64a0b1a45770648946b74799de2776b3fb8af42
parenta6920a25d98d8b126fe029620e0165d0d2ad15b2
BUG/MINOR: quic: Possible infinite loop in quic_build_post_handshake_frames()

This loop is due to the fact that we do not select the next node before
the conditional "continue" statement. Furthermore the condition and the
"continue" statement may be removed after replacing eb64_first() call
by eb64_lookup_ge(): we are sure this condition may not be satisfied.
Add some comments: this function initializes connection IDs with sequence
number 1 upto <max> non included.
Take the opportunity of this patch to remove a "return" wich broke this
traces rule: for any function, do not call TRACE_ENTER() without TRACE_LEAVE()!
Add also TRACE_ERROR() for any encoutered errors.

Must be backported to 2.6
src/xprt_quic.c