Hi Willy,

Here is a little patch to fix brackets place of the unlikely macro in http code. It must be backported in 1.8.

--
Christopher Faulet
>From fc53818025c8681c800e807960af2a6859006014 Mon Sep 17 00:00:00 2001
From: Christopher Faulet <[email protected]>
Date: Fri, 20 Jul 2018 09:54:26 +0200
Subject: [PATCH] BUG/MINOR: http: Set brackets for the unlikely macro at the
 right place

When test on the header "Early-Data" is made, the unlikely macro must encompass
the condition.

This patch must be backported in 1.8.
---
 src/proto_http.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/proto_http.c b/src/proto_http.c
index 117d27404..56035b9cb 100644
--- a/src/proto_http.c
+++ b/src/proto_http.c
@@ -3526,7 +3526,7 @@ int http_process_req_common(struct stream *s, struct channel *req, int an_bit, s
 		    ci_head(&s->req), &txn->hdr_idx, &ctx)) {
 			if (unlikely(http_header_add_tail2(&txn->req,
 			    &txn->hdr_idx, "Early-Data: 1",
-			    strlen("Early-Data: 1"))) < 0) {
+			    strlen("Early-Data: 1")) < 0)) {
 				goto return_bad_req;
 			 }
 		}
-- 
2.17.1

Reply via email to