From 6bb26d41fed6f0f19f9ecb76bbc034f185ee9b33 Mon Sep 17 00:00:00 2001 From: Christopher Faulet Date: Thu, 6 Apr 2023 08:58:42 +0200 Subject: [PATCH] BUG/MUNOR: http-ana: Use an unsigned integer for http_msg flags In the commit 2954bcc1e (BUG/MINOR: http-ana: Don't switch message to DATA when waiting for payload), the HTTP message flags were extended and don't fit anymore in an unsigned char. So, we must use an unsigned integer now. It is not a big deal because there was already a 6-bytes hole in the structure, just after the flags. Now, there are a 3-bytes hold before. This patch should fix the issue #2105. It is 2.8-specific, no backport needed. --- include/haproxy/http_ana-t.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/include/haproxy/http_ana-t.h b/include/haproxy/http_ana-t.h index 552f78ca0..1d6ab5899 100644 --- a/include/haproxy/http_ana-t.h +++ b/include/haproxy/http_ana-t.h @@ -224,8 +224,8 @@ enum h1_state { */ struct http_msg { enum h1_state msg_state; /* where we are in the current message parsing */ - unsigned char flags; /* flags describing the message (HTTP version, ...) */ - /* 5 bytes unused here */ + /* 3 bytes unused here */ + unsigned int flags; /* flags describing the message (HTTP version, ...) */ struct channel *chn; /* pointer to the channel transporting the message */ };