Article delegate-en/601 of [1-5169] on the server localhost:119
  upper oldest olders older1 this newer1 newers latest
search
[Top/Up] [oldest] - [Older+chunk] - [Newer+chunk] - [newest + Check]
[Reference:<_A600@delegate-en.ML_>]
Newsgroups: mail-lists.delegate-en

[DeleGate-En] Re: bug in FTP proxy when USER requires no password
14 Oct 1999 06:58:56 GMT ysato@etl.go.jp (Yutaka Sato)


Hi,

On 10/08/99(17:58) you Christoph Schwarz <plababdyi-uwzfqirv6zxr.ml@ml.delegate.org> wrote
in <_A600@delegate-en.ML_>
 |i found a bug in the FTP proxy (as of delegate 5.9.5):
 |When you connect to an account which requires no password, the FTP server sends
 |as reply to the USER command: 230 USER logged in. Delegate sends this 230
 |line back to the client, followed by 331 which means password required. But this
 |is incorrect, b/c login already succeeded. The client then sends PASS, but the
 |server returns an error message, so the client thinks login failed and because
 |of
 |this misunderstandig it is not possible to have an FTP session.
 |230 is a valid reply to a USER command, this means delegate does not implement
 |RFC correctly in this case.

Thank you for your notice.  I enclosed a patch which will fix the problem.

Cheers,
Yutaka
--
Yutaka Sato <ysato@etl.go.jp> http://www.etl.go.jp/~ysato/   @ @ 
Computer Science Division, Electrotechnical Laboratory      ( - )
1-1-4 Umezono, Tsukuba, Ibaraki, 305-8568 Japan            _<   >_

*** ../../delegate5.9.5/src/ftp.c	Tue Sep 14 08:35:38 1999
--- ftp.c	Thu Oct 14 15:51:24 1999
***************
*** 2965,2970 ****
--- 2965,2978 ----
  		arg[0] = 0;
  		if( dp = strstr(resp,"\n331") )
  			linescan(dp+1,arg,sizeof(arg));
+ 		else
+ 		if( scode == 331 ){
+ 			if( dp = strstr(resp,"\n230") ){
+ 				linescan(dp+1,com,sizeof(com));
+ 				sv1log("no password required: %s\n",com);
+ 				scode = 230;
+ 			}
+ 		}
  
  		if( strcasestr(arg," otp-") != NULL ){
  			fprintf(tc,"%s\r\n",arg);

  admin search upper oldest olders older1 this newer1 newers latest
[Top/Up] [oldest] - [Older+chunk] - [Newer+chunk] - [newest + Check]
@_@V