From cdb3594a33022ba59e21b75eebcfcfd39dccdf0f Mon Sep 17 00:00:00 2001
From: Fabian Keil <fk@fabiankeil.de>
Date: Tue, 14 Apr 2020 14:15:56 +0200
Subject: [PATCH] sed_https(): Update the last https header after running sed()
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit
This is necessary because addtional header may have been added.
Fixes a crash triggered by an assertion.
Reported by: Nedžad Hrnjica
Sponsored by: Robert Klemme
---
parsers.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/parsers.c b/parsers.c
index 45b4c3bf..70f368f1 100644
--- a/parsers.c
+++ b/parsers.c
@@ -1231,6 +1231,12 @@ jb_err sed_https(struct client_state *csp)
err = sed(csp, FILTER_CLIENT_HEADERS);
csp->flags |= CSP_FLAG_CLIENT_HEADER_PARSING_DONE;
+ /*
+ * Update the last header which may have changed
+ * due to header additions,
+ */
+ csp->https_headers->last = csp->headers->last;
+
csp->headers->first = headers.first;
csp->headers->last = headers.last;
--
2.49.0