diff options
Diffstat (limited to 'ext/ftp/php_ftp.c')
-rw-r--r-- | ext/ftp/php_ftp.c | 84 |
1 files changed, 41 insertions, 43 deletions
diff --git a/ext/ftp/php_ftp.c b/ext/ftp/php_ftp.c index aca0e0d934..4b3a0fb306 100644 --- a/ext/ftp/php_ftp.c +++ b/ext/ftp/php_ftp.c @@ -17,8 +17,6 @@ +----------------------------------------------------------------------+ */ -/* $Id$ */ - #ifdef HAVE_CONFIG_H #include "config.h" #endif @@ -126,7 +124,7 @@ ZEND_BEGIN_ARG_INFO(arginfo_ftp_systype, 0) ZEND_ARG_INFO(0, ftp) ZEND_END_ARG_INFO() -ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_fget, 0, 0, 4) +ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_fget, 0, 0, 3) ZEND_ARG_INFO(0, ftp) ZEND_ARG_INFO(0, fp) ZEND_ARG_INFO(0, remote_file) @@ -134,7 +132,7 @@ ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_fget, 0, 0, 4) ZEND_ARG_INFO(0, resumepos) ZEND_END_ARG_INFO() -ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_nb_fget, 0, 0, 4) +ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_nb_fget, 0, 0, 3) ZEND_ARG_INFO(0, ftp) ZEND_ARG_INFO(0, fp) ZEND_ARG_INFO(0, remote_file) @@ -147,7 +145,7 @@ ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_pasv, 0, 0, 2) ZEND_ARG_INFO(0, pasv) ZEND_END_ARG_INFO() -ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_get, 0, 0, 4) +ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_get, 0, 0, 3) ZEND_ARG_INFO(0, ftp) ZEND_ARG_INFO(0, local_file) ZEND_ARG_INFO(0, remote_file) @@ -155,7 +153,7 @@ ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_get, 0, 0, 4) ZEND_ARG_INFO(0, resume_pos) ZEND_END_ARG_INFO() -ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_nb_get, 0, 0, 4) +ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_nb_get, 0, 0, 3) ZEND_ARG_INFO(0, ftp) ZEND_ARG_INFO(0, local_file) ZEND_ARG_INFO(0, remote_file) @@ -167,7 +165,7 @@ ZEND_BEGIN_ARG_INFO(arginfo_ftp_nb_continue, 0) ZEND_ARG_INFO(0, ftp) ZEND_END_ARG_INFO() -ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_fput, 0, 0, 4) +ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_fput, 0, 0, 3) ZEND_ARG_INFO(0, ftp) ZEND_ARG_INFO(0, remote_file) ZEND_ARG_INFO(0, fp) @@ -175,7 +173,7 @@ ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_fput, 0, 0, 4) ZEND_ARG_INFO(0, startpos) ZEND_END_ARG_INFO() -ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_nb_fput, 0, 0, 4) +ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_nb_fput, 0, 0, 3) ZEND_ARG_INFO(0, ftp) ZEND_ARG_INFO(0, remote_file) ZEND_ARG_INFO(0, fp) @@ -183,7 +181,7 @@ ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_nb_fput, 0, 0, 4) ZEND_ARG_INFO(0, startpos) ZEND_END_ARG_INFO() -ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_put, 0, 0, 4) +ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_put, 0, 0, 3) ZEND_ARG_INFO(0, ftp) ZEND_ARG_INFO(0, remote_file) ZEND_ARG_INFO(0, local_file) @@ -191,14 +189,14 @@ ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_put, 0, 0, 4) ZEND_ARG_INFO(0, startpos) ZEND_END_ARG_INFO() -ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_append, 0, 0, 4) +ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_append, 0, 0, 3) ZEND_ARG_INFO(0, ftp) ZEND_ARG_INFO(0, remote_file) ZEND_ARG_INFO(0, local_file) ZEND_ARG_INFO(0, mode) ZEND_END_ARG_INFO() -ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_nb_put, 0, 0, 4) +ZEND_BEGIN_ARG_INFO_EX(arginfo_ftp_nb_put, 0, 0, 3) ZEND_ARG_INFO(0, ftp) ZEND_ARG_INFO(0, remote_file) ZEND_ARG_INFO(0, local_file) @@ -249,7 +247,7 @@ ZEND_END_ARG_INFO() /* }}} */ -const zend_function_entry php_ftp_functions[] = { +static const zend_function_entry php_ftp_functions[] = { PHP_FE(ftp_connect, arginfo_ftp_connect) #ifdef HAVE_FTP_SSL PHP_FE(ftp_ssl_connect, arginfo_ftp_ssl_connect) @@ -689,7 +687,7 @@ PHP_FUNCTION(ftp_alloc) ret = ftp_alloc(ftp, size, zresponse ? &response : NULL); if (response) { - zval_dtor(zresponse); + zval_ptr_dtor(zresponse); ZVAL_STR(zresponse, response); } @@ -824,7 +822,7 @@ PHP_FUNCTION(ftp_systype) } /* }}} */ -/* {{{ proto bool ftp_fget(resource stream, resource fp, string remote_file, int mode[, int resumepos]) +/* {{{ proto bool ftp_fget(resource stream, resource fp, string remote_file, [, int mode [, int resumepos]]) Retrieves a file from the FTP server and writes it to an open file */ PHP_FUNCTION(ftp_fget) { @@ -834,9 +832,9 @@ PHP_FUNCTION(ftp_fget) php_stream *stream; char *file; size_t file_len; - zend_long mode, resumepos=0; + zend_long mode=FTPTYPE_IMAGE, resumepos=0; - if (zend_parse_parameters(ZEND_NUM_ARGS(), "rrsl|l", &z_ftp, &z_file, &file, &file_len, &mode, &resumepos) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rrs|ll", &z_ftp, &z_file, &file, &file_len, &mode, &resumepos) == FAILURE) { return; } @@ -870,7 +868,7 @@ PHP_FUNCTION(ftp_fget) } /* }}} */ -/* {{{ proto int ftp_nb_fget(resource stream, resource fp, string remote_file, int mode[, int resumepos]) +/* {{{ proto int ftp_nb_fget(resource stream, resource fp, string remote_file [, int mode [, int resumepos]]) Retrieves a file from the FTP server asynchronly and writes it to an open file */ PHP_FUNCTION(ftp_nb_fget) { @@ -880,9 +878,9 @@ PHP_FUNCTION(ftp_nb_fget) php_stream *stream; char *file; size_t file_len; - zend_long mode, resumepos=0, ret; + zend_long mode=FTPTYPE_IMAGE, resumepos=0, ret; - if (zend_parse_parameters(ZEND_NUM_ARGS(), "rrsl|l", &z_ftp, &z_file, &file, &file_len, &mode, &resumepos) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rrs|ll", &z_ftp, &z_file, &file, &file_len, &mode, &resumepos) == FAILURE) { return; } @@ -944,7 +942,7 @@ PHP_FUNCTION(ftp_pasv) } /* }}} */ -/* {{{ proto bool ftp_get(resource stream, string local_file, string remote_file, int mode[, int resume_pos]) +/* {{{ proto bool ftp_get(resource stream, string local_file, string remote_file [, int mode [, int resume_pos]]) Retrieves a file from the FTP server and writes it to a local file */ PHP_FUNCTION(ftp_get) { @@ -954,9 +952,9 @@ PHP_FUNCTION(ftp_get) php_stream *outstream; char *local, *remote; size_t local_len, remote_len; - zend_long mode, resumepos=0; + zend_long mode=FTPTYPE_IMAGE, resumepos=0; - if (zend_parse_parameters(ZEND_NUM_ARGS(), "rppl|l", &z_ftp, &local, &local_len, &remote, &remote_len, &mode, &resumepos) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rpp|ll", &z_ftp, &local, &local_len, &remote, &remote_len, &mode, &resumepos) == FAILURE) { return; } @@ -1009,7 +1007,7 @@ PHP_FUNCTION(ftp_get) } /* }}} */ -/* {{{ proto int ftp_nb_get(resource stream, string local_file, string remote_file, int mode[, int resume_pos]) +/* {{{ proto int ftp_nb_get(resource stream, string local_file, string remote_file, [, int mode [, int resume_pos]]) Retrieves a file from the FTP server nbhronly and writes it to a local file */ PHP_FUNCTION(ftp_nb_get) { @@ -1020,9 +1018,9 @@ PHP_FUNCTION(ftp_nb_get) char *local, *remote; size_t local_len, remote_len; int ret; - zend_long mode, resumepos=0; + zend_long mode=FTPTYPE_IMAGE, resumepos=0; - if (zend_parse_parameters(ZEND_NUM_ARGS(), "rssl|l", &z_ftp, &local, &local_len, &remote, &remote_len, &mode, &resumepos) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rss|ll", &z_ftp, &local, &local_len, &remote, &remote_len, &mode, &resumepos) == FAILURE) { return; } @@ -1122,7 +1120,7 @@ PHP_FUNCTION(ftp_nb_continue) } /* }}} */ -/* {{{ proto bool ftp_fput(resource stream, string remote_file, resource fp, int mode[, int startpos]) +/* {{{ proto bool ftp_fput(resource stream, string remote_file, resource fp [, int mode [, int startpos]]) Stores a file from an open file to the FTP server */ PHP_FUNCTION(ftp_fput) { @@ -1130,11 +1128,11 @@ PHP_FUNCTION(ftp_fput) ftpbuf_t *ftp; ftptype_t xtype; size_t remote_len; - zend_long mode, startpos=0; + zend_long mode=FTPTYPE_IMAGE, startpos=0; php_stream *stream; char *remote; - if (zend_parse_parameters(ZEND_NUM_ARGS(), "rsrl|l", &z_ftp, &remote, &remote_len, &z_file, &mode, &startpos) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rsr|ll", &z_ftp, &remote, &remote_len, &z_file, &mode, &startpos) == FAILURE) { return; } @@ -1171,7 +1169,7 @@ PHP_FUNCTION(ftp_fput) } /* }}} */ -/* {{{ proto int ftp_nb_fput(resource stream, string remote_file, resource fp, int mode[, int startpos]) +/* {{{ proto int ftp_nb_fput(resource stream, string remote_file, resource fp [, int mode [, int startpos]]) Stores a file from an open file to the FTP server nbronly */ PHP_FUNCTION(ftp_nb_fput) { @@ -1180,11 +1178,11 @@ PHP_FUNCTION(ftp_nb_fput) ftptype_t xtype; size_t remote_len; int ret; - zend_long mode, startpos=0; + zend_long mode=FTPTYPE_IMAGE, startpos=0; php_stream *stream; char *remote; - if (zend_parse_parameters(ZEND_NUM_ARGS(), "rsrl|l", &z_ftp, &remote, &remote_len, &z_file, &mode, &startpos) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rsr|ll", &z_ftp, &remote, &remote_len, &z_file, &mode, &startpos) == FAILURE) { return; } @@ -1226,7 +1224,7 @@ PHP_FUNCTION(ftp_nb_fput) /* }}} */ -/* {{{ proto bool ftp_put(resource stream, string remote_file, string local_file, int mode[, int startpos]) +/* {{{ proto bool ftp_put(resource stream, string remote_file, string local_file [, int mode [, int startpos]]) Stores a file on the FTP server */ PHP_FUNCTION(ftp_put) { @@ -1235,10 +1233,10 @@ PHP_FUNCTION(ftp_put) ftptype_t xtype; char *remote, *local; size_t remote_len, local_len; - zend_long mode, startpos=0; + zend_long mode=FTPTYPE_IMAGE, startpos=0; php_stream *instream; - if (zend_parse_parameters(ZEND_NUM_ARGS(), "rppl|l", &z_ftp, &remote, &remote_len, &local, &local_len, &mode, &startpos) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rpp|ll", &z_ftp, &remote, &remote_len, &local, &local_len, &mode, &startpos) == FAILURE) { return; } @@ -1280,7 +1278,7 @@ PHP_FUNCTION(ftp_put) } /* }}} */ -/* {{{ proto bool ftp_append(resource stream, string remote_file, string local_file, int mode) +/* {{{ proto bool ftp_append(resource stream, string remote_file, string local_file [, int mode]) Append content of a file a another file on the FTP server */ PHP_FUNCTION(ftp_append) { @@ -1289,10 +1287,10 @@ PHP_FUNCTION(ftp_append) ftptype_t xtype; char *remote, *local; size_t remote_len, local_len; - zend_long mode; + zend_long mode=FTPTYPE_IMAGE; php_stream *instream; - if (zend_parse_parameters(ZEND_NUM_ARGS(), "rppl", &z_ftp, &remote, &remote_len, &local, &local_len, &mode) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rpp|l", &z_ftp, &remote, &remote_len, &local, &local_len, &mode) == FAILURE) { return; } @@ -1316,7 +1314,7 @@ PHP_FUNCTION(ftp_append) } /* }}} */ -/* {{{ proto int ftp_nb_put(resource stream, string remote_file, string local_file, int mode[, int startpos]) +/* {{{ proto int ftp_nb_put(resource stream, string remote_file, string local_file [, int mode [, int startpos]]) Stores a file on the FTP server */ PHP_FUNCTION(ftp_nb_put) { @@ -1325,10 +1323,10 @@ PHP_FUNCTION(ftp_nb_put) ftptype_t xtype; char *remote, *local; size_t remote_len, local_len; - zend_long mode, startpos=0, ret; + zend_long mode=FTPTYPE_IMAGE, startpos=0, ret; php_stream *instream; - if (zend_parse_parameters(ZEND_NUM_ARGS(), "rppl|l", &z_ftp, &remote, &remote_len, &local, &local_len, &mode, &startpos) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS(), "rpp|ll", &z_ftp, &remote, &remote_len, &local, &local_len, &mode, &startpos) == FAILURE) { return; } @@ -1543,7 +1541,7 @@ PHP_FUNCTION(ftp_set_option) switch (option) { case PHP_FTP_OPT_TIMEOUT_SEC: if (Z_TYPE_P(z_value) != IS_LONG) { - php_error_docref(NULL, E_WARNING, "Option TIMEOUT_SEC expects value of type long, %s given", + php_error_docref(NULL, E_WARNING, "Option TIMEOUT_SEC expects value of type int, %s given", zend_zval_type_name(z_value)); RETURN_FALSE; } @@ -1556,7 +1554,7 @@ PHP_FUNCTION(ftp_set_option) break; case PHP_FTP_OPT_AUTOSEEK: if (Z_TYPE_P(z_value) != IS_TRUE && Z_TYPE_P(z_value) != IS_FALSE) { - php_error_docref(NULL, E_WARNING, "Option AUTOSEEK expects value of type boolean, %s given", + php_error_docref(NULL, E_WARNING, "Option AUTOSEEK expects value of type bool, %s given", zend_zval_type_name(z_value)); RETURN_FALSE; } @@ -1565,7 +1563,7 @@ PHP_FUNCTION(ftp_set_option) break; case PHP_FTP_OPT_USEPASVADDRESS: if (Z_TYPE_P(z_value) != IS_TRUE && Z_TYPE_P(z_value) != IS_FALSE) { - php_error_docref(NULL, E_WARNING, "Option USEPASVADDRESS expects value of type boolean, %s given", + php_error_docref(NULL, E_WARNING, "Option USEPASVADDRESS expects value of type bool, %s given", zend_zval_type_name(z_value)); RETURN_FALSE; } |