Implement parser for Netware-style FTP LIST response listing.

net/ftp/ftp_directory_listing_parser_netware.cc

Index: net/ftp/ftp_directory_listing_parser_netware.cc
diff --git a/net/ftp/ftp_directory_listing_parser_netware.cc b/net/ftp/ftp_directory_listing_parser_netware.cc
new file mode 100644
index 0000000000000000000000000000000000000000..20d2b25308dd442f6d2f486af08e886d212f2061
--- /dev/null
+++ b/net/ftp/ftp_directory_listing_parser_netware.cc
@@ -0,0 +1,102 @@
+// 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_netware.h"
+
+#include <vector>
+
+#include "base/string_util.h"
+#include "net/ftp/ftp_util.h"
+
+namespace {
+
+bool LooksLikeNetwarePermissionsListing(const string16& text) {
+  if (text.length() != 10)
+    return false;
+
+  if (text[0] != '[' || text[9] != ']')
+    return false;
+  return (text[1] == 'R' || text[1] == '-') &&
+         (text[2] == 'W' || text[2] == '-') &&
+         (text[3] == 'C' || text[3] == '-') &&
+         (text[4] == 'E' || text[4] == '-') &&
+         (text[5] == 'A' || text[5] == '-') &&
+         (text[6] == 'F' || text[6] == '-') &&
+         (text[7] == 'M' || text[7] == '-') &&
+         (text[8] == 'S' || text[8] == '-');
+}
+
+}  // namespace
+
+namespace net {
+
+FtpDirectoryListingParserNetware::FtpDirectoryListingParserNetware()
+    : received_first_line_(false) {
+}
+
+bool FtpDirectoryListingParserNetware::ConsumeLine(const string16& line) {
+  if (!received_first_line_) {
+    received_first_line_ = true;
+
+    return StartsWith(line, ASCIIToUTF16("total "), true);
+  }
+
+  std::vector<string16> columns;
+  SplitString(CollapseWhitespace(line, false), ' ', &columns);
+
+  if (columns.size() != 8)
+    return false;
+
+  FtpDirectoryListingEntry entry;
+
+  if (columns[0].length() != 1)
+    return false;
+  if (columns[0][0] == 'd') {
+    entry.type = FtpDirectoryListingEntry::DIRECTORY;
+  } else if (columns[0][0] == '-') {
+    entry.type = FtpDirectoryListingEntry::FILE;
+  } else {
+    return false;
+  }
+
+  // Note: on older Netware systems the permissions listing is in the same
+  // column as the entry type (just there is no space between them). We do not
+  // support the older format here for simplicity.

[eroman, 2009/12/07 19:15:54]

is this a compat issue?

+  if (!LooksLikeNetwarePermissionsListing(columns[1]))
+    return false;
+
+  if (!StringToInt64(columns[3], &entry.size))
+    return false;
+  if (entry.size < 0)
+    return false;
+  if (entry.type != FtpDirectoryListingEntry::FILE)
+    entry.size = -1;
+
+  // Netware uses the same date listing format as Unix "ls -l".
+  if (!FtpUtil::LsDateListingToTime(columns[4], columns[5], columns[6],
+                                    &entry.last_modified)) {
+    return false;
+  }
+
+  entry.name = columns[7];
+
+  entries_.push(entry);
+  return true;
+}
+
+bool FtpDirectoryListingParserNetware::OnEndOfInput() {
+  return true;
+}
+
+bool FtpDirectoryListingParserNetware::EntryAvailable() const {
+  return !entries_.empty();
+}
+
+FtpDirectoryListingEntry FtpDirectoryListingParserNetware::PopEntry() {
+  FtpDirectoryListingEntry entry = entries_.front();

[eroman, 2009/12/07 19:15:54]

Might want to DCHECK(EntryAvailable())

+  entries_.pop();
+  return entry;
+}
+
+}  // namespace net