From d13a3c8d2ae5b9742c42db067705a6464f740af6 Mon Sep 17 00:00:00 2001
From: willy tarreau <willy@wtap.(none)>
Date: Mon, 10 Apr 2006 21:01:39 +0200
Subject: [PATCH] * ignore leading empty lines in HTTP requests (RFC2616)

---
 TODO      |  1 +
 haproxy.c | 26 ++++++++++++++++++++++++++
 2 files changed, 27 insertions(+)

diff --git a/TODO b/TODO
index ddc916634..1471c4b1b 100644
--- a/TODO
+++ b/TODO
@@ -149,4 +149,5 @@ Todo for 1.2
   activity on the buffer's pointers from touching the buffer page itself.
 - make buffer size configurable in global options
 * monitor number of simultaneous sessions in logs (per srv/inst/global)
+* ignore leading empty lines in HTTP requests
 
diff --git a/haproxy.c b/haproxy.c
index b8cc48d0e..aebf63e1e 100644
--- a/haproxy.c
+++ b/haproxy.c
@@ -3308,6 +3308,32 @@ int process_cli(struct session *t) {
 	    
 	    if (ptr == req->h) { /* empty line, end of headers */
 		int line, len;
+
+		/*
+		 * first, let's check that it's not a leading empty line, in
+		 * which case we'll ignore and remove it (according to RFC2616).
+		 */
+		if (req->h == req->data) {
+		    /* to get a complete header line, we need the ending \r\n, \n\r, \r or \n too */
+		    if (ptr > req->r - 2) {
+			/* this is a partial header, let's wait for more to come */
+			req->lr = ptr;
+			break;
+		    }
+
+		    /* now we know that *ptr is either \r or \n,
+		     * and that there are at least 1 char after it.
+		     */
+		    if ((ptr[0] == ptr[1]) || (ptr[1] != '\r' && ptr[1] != '\n'))
+			req->lr = ptr + 1; /* \r\r, \n\n, \r[^\n], \n[^\r] */
+		    else
+			req->lr = ptr + 2; /* \r\n or \n\r */
+		    /* ignore empty leading lines */
+		    buffer_replace2(req, req->h, req->lr, NULL, 0);
+		    req->h = req->lr;
+		    continue;
+		}
+
 		/* we can only get here after an end of headers */
 		/* we'll have something else to do here : add new headers ... */