]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUG/MINOR: mux-h2: Fix typo in scheme adjustment
authorTim Duesterhus <tim@bastelstu.be>
Sun, 28 Feb 2021 15:12:20 +0000 (16:12 +0100)
committerWilly Tarreau <w@1wt.eu>
Tue, 2 Mar 2021 13:13:57 +0000 (14:13 +0100)
That comma should've been a semicolon. Fortunately, as it is now there
is no impact thanks to operators precedence, and all expressions are
properly evaluated. But this is troubling and the risk is high to
turn it into an effective bug with a minor change.

Introduced in b8ce8905cf63ecd06b36af39c05103fadf3cc347 which first
appeared in 2.1-dev3. This fix must be backported to 2.1+.

src/mux_h2.c

index 4bffd452598ad5af44062e67b95179110d42f2c8..4dada0b0c664edc4fab7c9b603dc7abd1552d7b9 100644 (file)
@@ -5361,7 +5361,7 @@ static size_t h2s_bck_make_req_headers(struct h2s *h2s, struct htx *htx)
                        if (len + 2 < uri.len && uri.ptr[len + 1] == '/' && uri.ptr[len + 2] == '/') {
                                /* make the uri start at the authority now */
                                scheme.ptr = uri.ptr;
-                               scheme.len = len,
+                               scheme.len = len;
                                uri.ptr += len + 3;
                                uri.len -= len + 3;