From: wessels <>
Date: Fri, 30 Dec 2005 01:04:41 +0000 (+0000)
Subject: Found and fixed a case where HttpMsg::parse() returned false but
X-Git-Tag: SQUID_3_0_PRE4~408
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=7d5c64236c267684cf82a63607bc6c25791a449f;p=thirdparty%2Fsquid.git

Found and fixed a case where HttpMsg::parse() returned false but
forgot to set the error parameter.
---

diff --git a/src/HttpMsg.cc b/src/HttpMsg.cc
index 829ca962cc..53b9e00514 100644
--- a/src/HttpMsg.cc
+++ b/src/HttpMsg.cc
@@ -1,6 +1,6 @@
 
 /*
- * $Id: HttpMsg.cc,v 1.21 2005/12/01 18:15:51 wessels Exp $
+ * $Id: HttpMsg.cc,v 1.22 2005/12/29 18:04:41 wessels Exp $
  *
  * DEBUG: section 74    HTTP Message
  * AUTHOR: Alex Rousskov
@@ -168,8 +168,11 @@ bool HttpMsg::parse(MemBuf *buf, bool eof, http_status *error)
         return false;
     }
 
-    if (!sanityCheckStartLine(buf, error))	// redundant; could be remvoed
+    if (!sanityCheckStartLine(buf, error)) {
+        debugs(58,1, HERE << "first line of HTTP message is invalid");
+        *error = HTTP_INVALID_HEADER;
         return false;
+    }
 
     const int res = httpMsgParseStep(buf->content(), eof);