Fix FTP dir listing parser to support servers returning a "total n" line with an incorrect negative…

net/ftp/ftp_directory_listing_parser_ls.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.
 
 #include "net/ftp/ftp_directory_listing_parser_ls.h"
 
 #include <vector>
 
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_split.h"
@@ skipping 124 matching lines @@
         base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL);
 
     // Some FTP servers put a "total n" line at the beginning of the listing
     // (n is an integer). Allow such a line, but only once, and only if it's
     // the first non-empty line. Do not match the word exactly, because it may
     // be in different languages (at least English and German have been seen
     // in the field).
     if (columns.size() == 2 && !received_total_line) {
       received_total_line = true;
 
+      // Some FTP servers incorrectly return a negative integer for "n". Since
+      // this value is ignored anyway, just check any valid integer was
+      // provided.
       int64 total_number;
       if (!base::StringToInt64(columns[1], &total_number))
         return false;
-      if (total_number < 0)
-        return false;
 
       continue;
     }
 
     FtpDirectoryListingEntry entry;
 
     size_t column_offset;
     base::string16 size;
     if (!DetectColumnOffsetSizeAndModificationTime(columns,
                                                    current_time,
@@ skipping 61 matching lines @@
         entry.name = entry.name.substr(0, pos);
     }
 
     entries->push_back(entry);
   }
 
   return true;
 }
 
 }  // namespace net