Check in patch for pierre.lafayette, http://codereview.chromium.org/178059/sh...

net/url_request/url_request_new_ftp_job.cc

 // 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/url_request/url_request_new_ftp_job.h"
 
 #include "base/compiler_specific.h"
 #include "base/file_version_info.h"
 #include "base/message_loop.h"
 #include "base/sys_string_conversions.h"
@@ skipping 62 matching lines @@
 }
 
 URLRequestNewFtpJob::~URLRequestNewFtpJob() {
 }
 
 // static
 URLRequestJob* URLRequestNewFtpJob::Factory(URLRequest* request,
                                             const std::string& scheme) {
   DCHECK_EQ(scheme, "ftp");
 
+  int port = request->url().IntPort();
   if (request->url().has_port() &&
-      !net::IsPortAllowedByFtp(request->url().IntPort()))
+    !net::IsPortAllowedByFtp(port) && !net::IsPortAllowedByOverride(port))
     return new URLRequestErrorJob(request, net::ERR_UNSAFE_PORT);
 
   DCHECK(request->context());
   DCHECK(request->context()->ftp_transaction_factory());
   return new URLRequestNewFtpJob(request);
 }
 
 void URLRequestNewFtpJob::Start() {
   DCHECK(!transaction_.get());
   request_info_.url = request_->url();
@@ skipping 304 matching lines @@
   MessageLoop::current()->PostTask(FROM_HERE, NewRunnableMethod(
       this, &URLRequestNewFtpJob::OnStartCompleted, rv));
 }
 
 void URLRequestNewFtpJob::DestroyTransaction() {
   DCHECK(transaction_.get());
 
   transaction_.reset();
   response_info_ = NULL;
 }