diff --git a/net/ftp/ftp_directory_listing_parser_ls.cc b/net/ftp/ftp_directory_listing_parser_ls.cc index 38025a0d7677b403d75cc8d5d4dca3aa6129ed1b..038af116ccbb3bd95196e2a24d983bc9591dcc0b 100644 --- a/net/ftp/ftp_directory_listing_parser_ls.cc +++ b/net/ftp/ftp_directory_listing_parser_ls.cc @@ -158,9 +158,11 @@ bool FtpDirectoryListingParserLs::ConsumeLine(const string16& line) { entry.name = FtpUtil::GetStringPartAfterColumns(line, 6 + column_offset); if (entry.type == FtpDirectoryListingEntry::SYMLINK) { string16::size_type pos = entry.name.rfind(ASCIIToUTF16(" -> ")); - if (pos == string16::npos) - return false; - entry.name = entry.name.substr(0, pos); + + // We don't require the " -> " to be present. Some FTP servers don't send + // the symlink target, possibly for security reasons. + if (pos != string16::npos) + entry.name = entry.name.substr(0, pos); } entries_.push(entry); diff --git a/net/ftp/ftp_directory_listing_parser_ls_unittest.cc b/net/ftp/ftp_directory_listing_parser_ls_unittest.cc index 44c3eb612cddd3683d1569e19bad7b276d199402..2b57b403628bfbbadffddf626c166ccbfed8c9f3 100644 --- a/net/ftp/ftp_directory_listing_parser_ls_unittest.cc +++ b/net/ftp/ftp_directory_listing_parser_ls_unittest.cc @@ -49,6 +49,9 @@ TEST_F(FtpDirectoryListingParserLsTest, Good) { { "drwxrwxrwx 1 owner group 0 Sep 13 0:30 audio", net::FtpDirectoryListingEntry::DIRECTORY, "audio", -1, 1994, 9, 13, 0, 30 }, + { "lrwxrwxrwx 1 0 0 26 Sep 18 2008 pub", + net::FtpDirectoryListingEntry::SYMLINK, "pub", -1, + 2008, 9, 18, 0, 0 }, // Tests for the wu-ftpd variant: { "drwxr-xr-x 2 sys 512 Mar 27 2009 pub",