Commit 6421d69b authored by Daniel Stenberg's avatar Daniel Stenberg
Browse files

David McCreedy fixed libcurl to no longer ignore AUTH failures and now it 

reacts properly according to the CURLOPT_FTP_SSL setting.
parent 18081e30

CHANGES

+3 −0
Original line number Diff line number Diff line
@@ -7,6 +7,9 @@ 
                                  Changelog



Daniel (20 March 2006)

- David McCreedy fixed libcurl to no longer ignore AUTH failures and now it

  reacts properly according to the CURLOPT_FTP_SSL setting.



- Dan Fandrich fixed two TFTP problems: Fixed a bug whereby a received file

  whose length was a multiple of 512 bytes could have random garbage

  appended. Also, stop processing TFTP packets which are too short to be

RELEASE-NOTES

+2 −1
Original line number Diff line number Diff line
@@ -15,6 +15,7 @@ This release includes the following changes: 


This release includes the following bugfixes:



 o treat FTP AUTH failures properly

 o TFTP transfers could trash data

 o -d + -G combo crash
@@ -25,6 +26,6 @@ Other curl-related news since the previous public release: 
This release would not have looked like this without help, code, reports and

advice from friends like these:



 Dan Fandrich, Ilja van Sprundel

 Dan Fandrich, Ilja van Sprundel, David McCreedy



        Thanks! (and sorry if I forgot to mention someone)

lib/ftp.c

+8 −2
Original line number Diff line number Diff line
@@ -2430,8 +2430,14 @@ static CURLcode ftp_statemach_act(struct connectdata *conn) 
        result = Curl_nbftpsendf(conn, "AUTH %s", ftpauth[ftp->count1]);

        /* remain in this same state */

      }

      else {

        if(data->set.ftp_ssl > CURLFTPSSL_TRY)

          /* we failed and CURLFTPSSL_CONTROL or CURLFTPSSL_ALL is set */

          result = CURLE_FTP_SSL_FAILED;

        else

          /* ignore the failure and continue */

          result = ftp_state_user(conn);

      }



      if(result)

        return result;