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

[DeleGate-En] Re: FTP/SFTP Gateway PASV port re-use
03 Jun 2010 14:32:54 GMT Vijay Pandit <pyejabdyi-aipitibjyktr.ml@ml.delegate.org>


Hi Yutaka,

I applied the patch to *9.9.8-pre5* and compiled. Tested and it works fine!

Thank you for your support.

Cheers

Vijay


On Thu, Jun 3, 2010 at 2:28 AM, Yutaka Sato <feedback@delegate.org> wrote:

> Vijay,
>
> In message <_A4823@delegate-en.ML_>
> on 05/31/10(21:34:15)
> you Vijay Pandit <pyejabdyi-aipitibjyktr.ml@ml.delegate.org> wrote:
>  |I have attached the log file portion the last two successful transfers
> and
>  |the start of the failed transfers
>
> Thank you for the log file.  I found "----FIFO unlink(/tmp/dg-sftpgw/..."
> in your log file that caused the transmission error, and "errno=24" maybe
> for "Too many open files" to be cause of the FIFO error.
> There found two file descriptor leaks for each file transfer, thus the
> error will occur after the repetitive file transfers of counts around
> a half of the maximum number of file descriptors.
> I'll fix it as the enclosed patch in the next release.
>
> 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.9.8-pre5/src/sftp.c      Fri May 21 03:22:53 2010
> --- ./src/sftp.c        Thu Jun  3 01:20:29 2010
> ***************
> *** 1104,1109 ****
> --- 1104,1112 ----
>
>  relayFile2Fifo(fs,rfp,rpath,fifo);
>                                                _exit(0);
>                                          }
> +                                         else{
> +                                               fclose(rfp); /* 9.9.8 */
> +                                         }
>                                        }
>                                        putsftp(ts,"get %s %s",arg,rpath);
>                                }else{
> ***************
> *** 1135,1140 ****
> --- 1138,1146 ----
>                        }else{
>                                putresp(tc,500,"accept error");
>                        }
> +                       if( 0 <= vsock ){
> +                               close(vsock); /* 9.9.8 */
> +                       }
>                        vsock = -1;
>                }else
>                if( strcaseeq(com,"NLST") || strcaseeq(com,"LIST") ){
>


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