]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: h3/qpack: deal with too many headers
authorAmaury Denoyelle <adenoyelle@haproxy.com>
Tue, 14 Jun 2022 15:38:36 +0000 (17:38 +0200)
committerAmaury Denoyelle <adenoyelle@haproxy.com>
Wed, 15 Jun 2022 13:05:08 +0000 (15:05 +0200)
commit60ef19f137bad8cd97598970c708dd0bf4a89a70
tree85b2adfc1179e512c3995b74d7fa5493fbd59767
parent28d3c2489f2f6e159757f8daf37d097afaa2ef9b
BUG/MINOR: h3/qpack: deal with too many headers

ensures that we never insert too many entries in a headers input list.
On the decoding side, a new error QPACK_ERR_TOO_LARGE is reported in
this case.

This prevents crash if headers number on a H3 request or response is
superior to tune.http.maxhdr config value. Previously, a crash would
occur in QPACK decoding function.

Note that the process still crashes later with ABORT_NOW() because error
reporting on frame parsing is not implemented for now. It should be
treated with a RESET_STREAM frame in most cases.

This can be backported up to 2.6.
include/haproxy/qpack-dec.h
src/h3.c
src/qpack-dec.c