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
