BUG/MEDIUM: h1: Get the h1m state when restarting the headers parsing
Since the commit 0f8fb6b7f ("MINOR: h1: make the H1 headers block parser able to
parse headers only"), when headers are not received in one time, a parsing error
is returned because the local state in the function h1_headers_to_hdr_list() was
not initialized with the previous one (in fact, it was not initialized at all).
So now, we start the parsing of headers with the state H1_MSG_HDR_FIRST when the
flag H1_MF_HDRS_ONLY is set. Otherwise, we always get it from the h1m.
This patch must be backported to 1.9.
diff --git a/src/h1.c b/src/h1.c
index f9e8c9c..38d13c6 100644
--- a/src/h1.c
+++ b/src/h1.c
@@ -272,10 +272,11 @@
state = H1_MSG_HDR_FIRST;
h1m->next = 0;
}
- else if (h1m->state == H1_MSG_RQBEFORE || h1m->state == H1_MSG_RPBEFORE)
+ else {
state = h1m->state;
- else
- restarting = 1;
+ if (h1m->state != H1_MSG_RQBEFORE && h1m->state != H1_MSG_RPBEFORE)
+ restarting = 1;
+ }
ptr = start + h1m->next;
end = stop;