From cdb3594a33022ba59e21b75eebcfcfd39dccdf0f Mon Sep 17 00:00:00 2001 From: Fabian Keil 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.39.2