Re: cvs: php4 /ext/ftp ftp.c

I just did. Sorry - it took a while - my tree was a bit too outdated, and my connection sucks :( Vlad derick@php.net wrote:

Hey Vlad, are you going to merge it, or should I do it? Derick On Mon, 18 Mar 2002, Vlad Krupin wrote:

vlad Mon Mar 18 17:26:33 2002 EDT

 Modified files:                 /php4/ext/ftp	ftp.c  Log:

Fix ftp_size() returning bogus results. # make sure the server is in BINARY mode before requesting file size. Index: php4/ext/ftp/ftp.c diff -u php4/ext/ftp/ftp.c:1.48 php4/ext/ftp/ftp.c:1.49 --- php4/ext/ftp/ftp.c:1.48 Mon Mar 18 13:54:24 2002 +++ php4/ext/ftp/ftp.c Mon Mar 18 17:26:32 2002 @@ -16,7 +16,7 @@ +----------------------------------------------------------------------+ */ -/* $Id: ftp.c,v 1.48 2002/03/18 18:54:24 wez Exp $ */ +/* $Id: ftp.c,v 1.49 2002/03/18 22:26:32 vlad Exp $ */ #include "php.h" @@ -685,6 +685,8 @@ if (ftp == NULL) return -1; + if (!ftp_type(ftp, FTPTYPE_IMAGE)) + return -1; if (!ftp_putcmd(ftp, "SIZE", path)) return -1; if (!ftp_getresp(ftp) || ftp->resp != 213) -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php

-----------------------------------------------------------------------

             PHP: Scripting the Web - derick@php.net
               All your branches are belong to me!
           SRM: Script Running Machine - www.vl-srm.net

-----------------------------------------------------------------------