Fix a bug with parsing ftp directory listing lines.

net/ftp/ftp_directory_listing_parser_windows.cc

 // Copyright (c) 2009 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.
 
 #include "net/ftp/ftp_directory_listing_parser_windows.h"
 
 #include <vector>
 
 #include "base/string_util.h"
 #include "net/ftp/ftp_util.h"
@@ skipping 30 matching lines @@
   if (columns[1].length() != 7)
     return false;
   std::vector<string16> time_parts;
   SplitString(columns[1].substr(0, 5), ':', &time_parts);
   if (time_parts.size() != 2)
     return false;
   if (!StringToInt(time_parts[0], &time_exploded.hour))
     return false;
   if (!StringToInt(time_parts[1], &time_exploded.minute))
     return false;
+  if (!time_exploded.HasValidValues())
+    return false;
   string16 am_or_pm(columns[1].substr(5, 2));
-  if (EqualsASCII(am_or_pm, "PM"))
-    time_exploded.hour += 12;
-  else if (!EqualsASCII(am_or_pm, "AM"))
+  if (EqualsASCII(am_or_pm, "PM")) {
+    if (time_exploded.hour < 12)
+      time_exploded.hour += 12;
+  } else if (EqualsASCII(am_or_pm, "AM")) {
+    if (time_exploded.hour == 12)
+      time_exploded.hour = 0;
+  } else {
     return false;
+  }
 
   // We don't know the time zone of the server, so just use local time.
   *time = base::Time::FromLocalExploded(time_exploded);
   return true;
 }
 
 }  // namespace
 
 namespace net {
 
@@ skipping 39 matching lines @@
   return !entries_.empty();
 }
 
 FtpDirectoryListingEntry FtpDirectoryListingParserWindows::PopEntry() {
   FtpDirectoryListingEntry entry = entries_.front();
   entries_.pop();
   return entry;
 }
 
 }  // namespace net