Article delegate-en/4006 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:<_A4005@delegate-en.ML_>]
Newsgroups: mail-lists.delegate-en

[DeleGate-En] Re: delegate 9.7.7 and io timeout
03 Jul 2008 06:41:53 GMT feedback@delegate.org (Yutaka Sato)
The DeleGate Project


Hi,

In message <_A4005@delegate-en.ML_> on 07/03/08(10:48:06)
you puuhqbdyi-ry4zqclwjzvr.ml@ml.delegate.org wrote:
 |Few days ago, I updated delegate from 8.5.9 to 9.7.7.
 |Then I found that ftp timeout (TIMEOUT=io:0) seems not working correctly.
 |
 |In delegate log , I could find "poll-TIMEOUT" error.Please see belows,
...
 |07/03 10:01:09.17 [14160] 5+0/6/4: ftp_conndata: connected 
 |172.31.92.46:47671->172.31.145.174/172.31.145.174:1750 [24](0.0)
 |07/03 10:01:09.17 [14160] 5+0/6/4: DATA xx.xx.xx.xx:9291 -> 
 |172.31.92.46:47657 .. 172.31.92.46:47671 -> 172.31.145.174:1750
 |07/03 10:01:09.17 [14160] 5+0/6/4: FTP data-relay([23]15554b -> 
 |[24]10000b) 0b / 0/ (0) 0.00s (poll-TIMEOUT)
 |
 |When I changed timeout value to default(600sec), 9.7.7 also works well.
 |I'm not sure it's bug or not. "0" does not mean "never timeout" no longer?

I confirmed FTP-DeleGate acts like you described since 9.2.2-pre1 in which
I supported the abortion of the data transfer on FTP (ABOR command).
It must be called a bug and will be fixed as the enclosed patch, but also
I feel it not practical to to set the timeout value to infinite :)

Cheers,
Yutaka
--
  9 9   Yutaka Sato <y.sato@delegate.org> http://delegate.org/y.sato/
 ( ~ )  National Institute of Advanced Industrial Science and Technology
_<   >_ 1-1-4 Umezono, Tsukuba, Ibaraki, 305-8568 Japan
Do the more with the less -- B. Fuller


*** dist/src/delegate9.8.2-pre45/src/ftp.c	Tue Jun 17 13:25:02 2008
--- src/ftp.c	Thu Jul  3 15:38:33 2008
***************
*** 4284,4289 ****
--- 4284,4293 ----
  	ofdv[0] = dst;   oqev[0] = PS_OUT;
  	ofdv[1] = FromC; oqev[1] = PS_IN|PS_PRI;
  	timeout = IO_TIMEOUT * 1000;
+ 	if( timeout == 0 ){
+ 		timeout = -1;
+ 		/* the value for infinite timeout in PollInsOuts() */
+ 	}
  
  	for( ngets = 0; ; ngets++  ){
  		if( FCF.fc_chokedata ){

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