BUG/MEDIUM: http: functions set-{path,query,method,uri} breaks the HTTP parser

When one of these functions replaces a part of the query string by
a shorter or longer new one, the header parsing is broken. This is
because the start of the first header is not updated.

In the same way, the total length of the request line is not updated.
I dont see any bug caused by this miss, but I guess than it is better
to store the good length.

This bug is only in the development version.

diff --git a/src/proto_http.c b/src/proto_http.c
index d20225a921325f13f32b7292470b5a8f405f8967..6bb0745eecdd4a780d105928d71f0f2ddf0514d1 100644 (file)
--- a/src/proto_http.c
+++ b/src/proto_http.c
@@ -11825,6 +11825,8 @@ int http_replace_req_line(int action, const char *replace, int len,
 
        /* commit changes and adjust end of message */
        delta = buffer_replace2(s->req.buf, cur_ptr, cur_end, replace + offset, len - offset);
+       txn->req.sl.rq.l += delta;
+       txn->hdr_idx.v[0].len += delta;
        http_msg_move_end(&txn->req, delta);
        return 0;
 }