Modified to resolve TODO in parseversion in http_response_headers.cc

net/http/http_response_headers.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // The rules for header parsing were borrowed from Firefox:
 // http://lxr.mozilla.org/seamonkey/source/netwerk/protocol/http/src/nsHttpResponseHead.cpp
 // The rules for parsing content-types were also borrowed from Firefox:
 // http://lxr.mozilla.org/mozilla/source/netwerk/base/src/nsURLHelper.cpp#834
 
 #include "net/http/http_response_headers.h"
@@ skipping 602 matching lines @@
 bool HttpResponseHeaders::HasHeader(const base::StringPiece& name) const {
   return FindHeader(0, name) != std::string::npos;
 }
 
 HttpResponseHeaders::HttpResponseHeaders() : response_code_(-1) {
 }
 
 HttpResponseHeaders::~HttpResponseHeaders() {
 }
 
+// Function to parse HTTP Version in accordance to
+// RFC2616 sec 3.1: HTTP-Version = "HTTP" "/" 1*DIGIT "." 1*DIGIT
+// if input string is valid, returns corresponding number in value
+// else returns false
+bool HttpResponseHeaders::ParseVersionInternal(
+    std::string::const_iterator begin,
+    std::string::const_iterator end,
+    uint32* value) {
+  std::string error_msg;
+  if(!base::StringToUint(StringPiece(begin, end), value)) {
+    if(*value == 0) {
+      if(begin == end) {
+        error_msg = "Empty String!!!";
+      } else if(!(*begin >= '0' && *begin <= '9')) {
+          error_msg = "Invalid char at beginning!!!";
+      } else {
+          error_msg = "Underflow!!!";
+      }
+    } else if(*value == 0xFFFFFFFF) {
+        error_msg = "Overflow!!!";
+    } else if(!(*(end-1) >= '0' && *(end-1) <= '9')) {
+        error_msg = "Invalid char at end!!!";
+    } else {
+        error_msg = "Invalid value!!!";
+    }
+    DVLOG(1) << error_msg;
+    return false;
+  }
+  return true;
+}
+
 // Note: this implementation implicitly assumes that line_end points at a valid
 // sentinel character (such as '\0').
 // static
 HttpVersion HttpResponseHeaders::ParseVersion(
     std::string::const_iterator line_begin,
     std::string::const_iterator line_end) {
   std::string::const_iterator p = line_begin;
 
-  // RFC2616 sec 3.1: HTTP-Version   = "HTTP" "/" 1*DIGIT "." 1*DIGIT
-  // TODO: (1*DIGIT apparently means one or more digits, but we only handle 1).
-  // TODO: handle leading zeros, which is allowed by the rfc1616 sec 3.1.
-
   if ((line_end - p < 4) || !LowerCaseEqualsASCII(p, p + 4, "http")) {
     DVLOG(1) << "missing status line";
     return HttpVersion();
   }
 
   p += 4;
 
   if (p >= line_end || *p != '/') {
     DVLOG(1) << "missing version";
     return HttpVersion();
   }
 
   std::string::const_iterator dot = std::find(p, line_end, '.');
-  if (dot == line_end) {
-    DVLOG(1) << "malformed version";
+  std::string::const_iterator end_of_minor = dot+1;
+    
+  // find end of minor.
+  while(*end_of_minor != ' ')
+    end_of_minor++;
+  
+  ++p;  // from / to first digit.
+  uint32 major = 0;
+  uint32 minor = 0;
+
+  if (ParseVersionInternal(p, dot, &major) &&
+      ParseVersionInternal(dot + 1, end_of_minor, &minor)) {
+    DVLOG(1) << "ParseVersion success!!!";
+    return HttpVersion(major, minor);
+  } else {
+    DVLOG(1) << "ParseVersion fail!!!";
     return HttpVersion();
   }
-
-  ++p;  // from / to first digit.
-  ++dot;  // from . to second digit.
-
-  if (!(*p >= '0' && *p <= '9' && *dot >= '0' && *dot <= '9')) {
-    DVLOG(1) << "malformed version number";
-    return HttpVersion();
-  }
-
-  uint16 major = *p - '0';
-  uint16 minor = *dot - '0';
-
-  return HttpVersion(major, minor);
 }
 
 // Note: this implementation implicitly assumes that line_end points at a valid
 // sentinel character (such as '\0').
 void HttpResponseHeaders::ParseStatusLine(
     std::string::const_iterator line_begin,
     std::string::const_iterator line_end,
     bool has_headers) {
   // Extract the version number
   parsed_http_version_ = ParseVersion(line_begin, line_end);
 
-  // Clamp the version number to one of: {0.9, 1.0, 1.1}
-  if (parsed_http_version_ == HttpVersion(0, 9) && !has_headers) {
-    http_version_ = HttpVersion(0, 9);
-    raw_headers_ = "HTTP/0.9";
-  } else if (parsed_http_version_ >= HttpVersion(1, 1)) {
-    http_version_ = HttpVersion(1, 1);
-    raw_headers_ = "HTTP/1.1";
-  } else {
-    // Treat everything else like HTTP 1.0
+  //Allow mulitple digits in major and minor values of HTTP Version.
+  //If parseVersion failed, keep version as 1.0 as per old logic
+  //If version is 0.9 and has headers, keep version as 1.0 as per old logic
+  //If version is 0.9 and has no headers, keep version as 0.9 as per old logic
+  //Else in all other cases, http_version will be same as parsed_http_version
+  if (parsed_http_version_ == HttpVersion()) {
     http_version_ = HttpVersion(1, 0);
     raw_headers_ = "HTTP/1.0";
+  } else if (parsed_http_version_ == HttpVersion(0, 9) && has_headers) {
+    http_version_ = HttpVersion(1, 0);
+    raw_headers_ = "HTTP/1.0";
+  } else {
+    http_version_ = parsed_http_version_;
+    raw_headers_ = "HTTP/";
+    raw_headers_.append(base::UintToString(http_version_.major_value()));
+    raw_headers_.append(".");
+    raw_headers_.append(base::UintToString(http_version_.minor_value()));
   }
   if (parsed_http_version_ != http_version_) {
     DVLOG(1) << "assuming HTTP/" << http_version_.major_value() << "."
              << http_version_.minor_value();
   }
 
   // TODO(eroman): this doesn't make sense if ParseVersion failed.
   std::string::const_iterator p = std::find(line_begin, line_end, ' ');
 
   if (p == line_end) {
@@ skipping 700 matching lines @@
   return true;
 }
 
 bool HttpResponseHeaders::IsChunkEncoded() const {
   // Ignore spurious chunked responses from HTTP/1.0 servers and proxies.
   return GetHttpVersion() >= HttpVersion(1, 1) &&
       HasHeaderValue("Transfer-Encoding", "chunked");
 }
 
 }  // namespace net