Re: CCC and RFC4217 compliance?

"Alexander V. Lukyanov" <[email protected]> Thu, 18 Feb 2010 17:40:46 +0300
Newsgroups gmane.network.lftp.devel
Message-ID <[email protected]>
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();
       }

--
   Alexander.