FTP: work around server bugs resulting in negative file size being sent.

net/ftp/ftp_directory_listing_parser_ls.cc

 // Copyright (c) 2011 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/string_number_conversions.h"
 #include "base/string_split.h"
@@ skipping 169 matching lines @@
 
     if (!base::StringToInt64(columns[column_offset - 3], &entry.size)) {
       // Some FTP servers do not separate owning group name from file size,
       // like "group1234". We still want to display the file name for that
       // entry, but can't really get the size (What if the group is named
       // "group1", and the size is in fact 234? We can't distinguish between
       // that and "group" with size 1234). Use a dummy value for the size.
       // TODO(phajdan.jr): Use a value that means "unknown" instead of 0 bytes.
       entry.size = 0;
     }
-    if (entry.size < 0)
-      return false;
+    if (entry.size < 0) {
+      // Some FTP servers have bugs that cause them to display the file size
+      // as negative. They're most likely big files like DVD ISO images.
+      // We still want to display them, so just say the real file size
+      // is unknown.
+      entry.size = -1;
+    }
     if (entry.type != FtpDirectoryListingEntry::FILE)
       entry.size = -1;
 
     if (column_offset == columns.size() - 1) {
       // If the end of the date listing is the last column, there is no file
       // name. Some FTP servers send listing entries with empty names.
       // It's not obvious how to display such an entry, so we ignore them.
       // We don't want to make the parsing fail at this point though.
       // Other entries can still be useful.
       continue;
@@ skipping 11 matching lines @@
         entry.name = entry.name.substr(0, pos);
     }
 
     entries->push_back(entry);
   }
 
   return true;
 }
 
 }  // namespace net