Bug fix for lack of errors on requests with invalid ports

BUG=http://code.google.com/p/chromium/issues/detail?id=138735


Review URL: https://chromiumcodereview.appspot.com/10834053

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148925 0039d316-1c4b-4281-b951-d872f2087c98

net/url_request/ftp_protocol_handler.cc

@@ -5,7 +5,12 @@
 #include "net/url_request/ftp_protocol_handler.h"
 
 #include "base/logging.h"
+#include "net/base/net_errors.h"
+#include "net/base/net_util.h"
+#include "net/url_request/url_request.h"
+#include "net/url_request/url_request_error_job.h"
 #include "net/url_request/url_request_ftp_job.h"
+#include "googleurl/src/gurl.h"
 
 namespace net {
 
@@ -22,6 +27,12 @@ FtpProtocolHandler::FtpProtocolHandler(
 
 URLRequestJob* FtpProtocolHandler::MaybeCreateJob(
     URLRequest* request) const {
+  int port = request->url().IntPort();
+  if (request->url().has_port() &&
+      !IsPortAllowedByFtp(port) && !IsPortAllowedByOverride(port)) {
+    return new URLRequestErrorJob(request, ERR_UNSAFE_PORT);
+  }
+
   return new URLRequestFtpJob(request,
                               network_delegate_,
                               ftp_transaction_factory_,