* Let the ftp method work with ftp servers which do not...

Author: mdz
Date: 2003-08-10 02:24:39 GMT
* Let the ftp method work with ftp servers which do not require a
password (Closes: #199425)

diff --git a/debian/changelog b/debian/changelog
index 4fc61a8c6293a62ada1d7fe91c085d530fa672d0..21dd8f634de270e24120891256e0f294ac4e0feb 100644 (file)
--- a/debian/changelog
+++ b/debian/changelog
@@ -49,6 +49,8 @@ apt (0.5.9) unstable; urgency=low
     - echo 3 > debian/compat
     - Build-Depends: debhelper (>= 3)
   * Exclude '.#*' from cvs-build
+  * Let the ftp method work with ftp servers which do not require a
+    password (Closes: #199425)
 
  --
 

diff --git a/methods/ftp.cc b/methods/ftp.cc
index cc2f1395d92bfdbffa3155594139ea43db8e091b..fb84f3f5553fe04ea3472bd0e412bb68ba1dd630 100644 (file)
--- a/methods/ftp.cc
+++ b/methods/ftp.cc
@@ -1,6 +1,6 @@
 // -*- mode: cpp; mode: fold -*-
 // Description                                                         /*{{{*/
-// $Id: ftp.cc,v 1.30 2003/02/10 07:34:41 doogie Exp $
+// $Id: ftp.cc,v 1.31 2003/08/10 02:24:39 mdz Exp $
 /* ######################################################################
 
    FTP Aquire Method - This is the FTP aquire method for APT.
@@ -209,11 +209,13 @@ bool FTPConn::Login()
       if (Tag >= 400)
         return _error->Error(_("USER failed, server said: %s"),Msg.c_str());
       
-      // Send the Password
-      if (WriteMsg(Tag,Msg,"PASS %s",Pass.c_str()) == false)
-        return false;
-      if (Tag >= 400)
-        return _error->Error(_("PASS failed, server said: %s"),Msg.c_str());
+      if (Tag == 331) { // 331 User name okay, need password.
+         // Send the Password
+         if (WriteMsg(Tag,Msg,"PASS %s",Pass.c_str()) == false)
+            return false;
+         if (Tag >= 400)
+            return _error->Error(_("PASS failed, server said: %s"),Msg.c_str());
+      }
       
       // Enter passive mode
       if (_config->Exists("Acquire::FTP::Passive::" + ServerName.Host) == true)