diff options
| author | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-08 18:20:57 +0000 |
|---|---|---|
| committer | phajdan.jr@chromium.org <phajdan.jr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-07-08 18:20:57 +0000 |
| commit | 39de121a7e81a1f399a1e83fee97591383546c88 (patch) | |
| tree | ae2a65f14d20181758c459deea9003394c6924a0 /net/ftp/ftp_network_transaction.h | |
| parent | 1f4c7b31ed9395dac76ffd5e1efa95223291492a (diff) | |
| download | chromium_src-39de121a7e81a1f399a1e83fee97591383546c88.zip chromium_src-39de121a7e81a1f399a1e83fee97591383546c88.tar.gz chromium_src-39de121a7e81a1f399a1e83fee97591383546c88.tar.bz2 | |
FTP: change order of commands to improve compatibility
It turns out some FTP servers react strangely to a RETR for an
unexistent file. They send a success reponse followed by a failure
response.
The solution is to move the file/directory sniffing logic from
RETR earlier, to SIZE.
BUG=44582
TEST=net_unittests
Review URL: http://codereview.chromium.org/2813044
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@51866 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/ftp/ftp_network_transaction.h')
| -rw-r--r-- | net/ftp/ftp_network_transaction.h | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/net/ftp/ftp_network_transaction.h b/net/ftp/ftp_network_transaction.h index d80e564..23a53cbc 100644 --- a/net/ftp/ftp_network_transaction.h +++ b/net/ftp/ftp_network_transaction.h @@ -61,7 +61,6 @@ class FtpNetworkTransaction : public FtpTransaction { COMMAND_CWD, COMMAND_MLSD, COMMAND_LIST, - COMMAND_MDTM, COMMAND_QUIT, }; @@ -179,8 +178,6 @@ class FtpNetworkTransaction : public FtpTransaction { int ProcessResponseMLSD(const FtpCtrlResponse& response); int DoCtrlWriteLIST(); int ProcessResponseLIST(const FtpCtrlResponse& response); - int DoCtrlWriteMDTM(); - int ProcessResponseMDTM(const FtpCtrlResponse& response); int DoCtrlWriteQUIT(); int ProcessResponseQUIT(const FtpCtrlResponse& response); @@ -276,7 +273,6 @@ class FtpNetworkTransaction : public FtpTransaction { STATE_CTRL_WRITE_CWD, STATE_CTRL_WRITE_MLSD, STATE_CTRL_WRITE_LIST, - STATE_CTRL_WRITE_MDTM, STATE_CTRL_WRITE_QUIT, // Data connection states: STATE_DATA_CONNECT, |