On Thu, Feb 18, 2010 at 09:23:48AM -0800, Ray Van Dolson wrote: > On Thu, Feb 18, 2010 at 06:40:46AM -0800, Alexander V. Lukyanov wrote: > > On Wed, Feb 17, 2010 at 05:27:40PM -0800, Ray Van Dolson wrote: > > > It sounds like lftp doesn't properly support a bi-directional SSL > > > shutdown when CCC is in use as described in RFC4217 section 12.3. Is > > > this the case? > > > > Would this patch fix the problem? I have no server to test this on. > > > > Index: ftpclass.cc > > =================================================================== > > RCS file: /home/lav/cvsroot/lftp/src/ftpclass.cc,v > > retrieving revision 1.461 > > diff -u -p -r1.461 ftpclass.cc > > --- ftpclass.cc 15 Sep 2009 07:56:30 -0000 1.461 > > +++ ftpclass.cc 18 Feb 2010 14:27:29 -0000 > > @@ -4136,6 +4148,7 @@ void Ftp::CheckResp(int act) > > case Expect::CCC: > > if(is2XX(act)) > > { > > + conn->control_send->PutEOF(); > > state=WAITING_CCC_SHUTDOWN; > > conn->waiting_ssl_timer.Reset(); > > } > > > > That does seem to work.... still testing, but so far so good. > > (Tested against lftp 3.7.14 w/ Fedora 11) >
Also works with lftp 3.7.11 on RHEL 5.4. These tests are all being done against a ProFTPD 1.3.1 server with mod_tls enabled. Ray