BUG=1112

net/url_request/url_request_ftp_job.cc

 // Copyright (c) 2006-2008 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/url_request/url_request_ftp_job.h"
 
 #include <windows.h>
 #include <wininet.h>
 
 #include "base/message_loop.h"
@@ skipping 205 matching lines @@
       }
       default:
         CleanupConnection();
         NotifyStartError(URLRequestStatus(URLRequestStatus::FAILED,
                          WinInetUtil::OSErrorToNetError(result.dwError)));
         return;
     }
   } else if (state_ == SETTING_CUR_DIRECTORY) {
     OnSetCurrentDirectory(result.dwError);
   } else if (state_ == FINDING_FIRST_FILE) {
-    if (result.dwError != ERROR_SUCCESS) {
+    // We don't fail here if result.dwError != ERROR_SUCCESS because
+    // getting an error here doesn't always mean the file is not found.
+    // FindFirstFileA() issue a LIST command and may fail on some
+    // ftp server when the requested object is a file. So ERROR_NO_MORE_FILES
+    // from FindFirstFileA() is not a reliable criteria for valid path
+    // or not, we should proceed optimistically by getting the file handle.
+    if (result.dwError != ERROR_SUCCESS &&
+        result.dwError != ERROR_NO_MORE_FILES) {
       DWORD result_error = result.dwError;
       CleanupConnection();
-      // Fixup the error message from our directory/file guessing.
-      if (!is_directory_ && result_error == ERROR_NO_MORE_FILES)
-        result_error = ERROR_PATH_NOT_FOUND;
       NotifyStartError(URLRequestStatus(URLRequestStatus::FAILED,
                        WinInetUtil::OSErrorToNetError(result_error)));
       return;
     }
     request_handle_ = (HINTERNET)result.dwResult;
     OnFindFirstFile(result.dwError);
   } else if (state_ == GETTING_DIRECTORY) {
     OnFindFile(result.dwError);
   } else if (state_ == GETTING_FILE_HANDLE) {
     if (result.dwError != ERROR_SUCCESS) {
@@ skipping 277 matching lines @@
 
       *location = request_->url().ReplaceComponents(replacements);
       *http_status_code = 301;  // simulate a permanent redirect
       return true;
     }
   }
 
   return false;
 }