diff options
Diffstat (limited to 'ext')
| -rw-r--r-- | ext/curl/curl_file.c | 32 | ||||
| -rw-r--r-- | ext/curl/interface.c | 71 | ||||
| -rw-r--r-- | ext/curl/multi.c | 69 | ||||
| -rw-r--r-- | ext/curl/share.c | 26 | 
4 files changed, 106 insertions, 92 deletions
| diff --git a/ext/curl/curl_file.c b/ext/curl/curl_file.c index 10a5ae55e7..888b884138 100644 --- a/ext/curl/curl_file.c +++ b/ext/curl/curl_file.c @@ -31,24 +31,24 @@ PHP_CURL_API zend_class_entry *curl_CURLFile_class;  static void curlfile_ctor(INTERNAL_FUNCTION_PARAMETERS)  { -	char *fname = NULL, *mime = NULL, *postname = NULL; -	size_t fname_len, mime_len, postname_len; +	zend_string *fname, *mime = NULL, *postname = NULL;  	zval *cf = return_value; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "s|ss", &fname, &fname_len, &mime, &mime_len, &postname, &postname_len) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(1,3) +		Z_PARAM_STR(fname) +		Z_PARAM_OPTIONAL +		Z_PARAM_STR(mime) +		Z_PARAM_STR(postname) +	ZEND_PARSE_PARAMETERS_END(); -	if (fname) { -		zend_update_property_string(curl_CURLFile_class, cf, "name", sizeof("name")-1, fname); -	} +	zend_update_property_string(curl_CURLFile_class, cf, "name", sizeof("name")-1, ZSTR_VAL(fname));  	if (mime) { -		zend_update_property_string(curl_CURLFile_class, cf, "mime", sizeof("mime")-1, mime); +		zend_update_property_string(curl_CURLFile_class, cf, "mime", sizeof("mime")-1, ZSTR_VAL(mime));  	}  	if (postname) { -		zend_update_property_string(curl_CURLFile_class, cf, "postname", sizeof("postname")-1, postname); +		zend_update_property_string(curl_CURLFile_class, cf, "postname", sizeof("postname")-1, ZSTR_VAL(postname));  	}  } @@ -84,13 +84,13 @@ static void curlfile_get_property(char *name, size_t name_len, INTERNAL_FUNCTION  static void curlfile_set_property(char *name, size_t name_len, INTERNAL_FUNCTION_PARAMETERS)  { -	char *arg = NULL; -	size_t arg_len; +	zend_string *arg; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "s", &arg, &arg_len) == FAILURE) { -		return; -	} -	zend_update_property_string(curl_CURLFile_class, getThis(), name, name_len, arg); +	ZEND_PARSE_PARAMETERS_START(1,1) +		Z_PARAM_STR(arg) +	ZEND_PARSE_PARAMETERS_END(); + +	zend_update_property_string(curl_CURLFile_class, getThis(), name, name_len, ZSTR_VAL(arg));  }  /* {{{ proto string CURLFile::getFilename() diff --git a/ext/curl/interface.c b/ext/curl/interface.c index 4595f64064..e5d6d432a2 100644 --- a/ext/curl/interface.c +++ b/ext/curl/interface.c @@ -1964,12 +1964,12 @@ PHP_FUNCTION(curl_init)  {  	php_curl *ch;  	CURL 	 *cp; -	char 	 *url = NULL; -	size_t		  url_len = 0; +	zend_string *url = NULL; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "|s", &url, &url_len) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(0,1) +		Z_PARAM_OPTIONAL +		Z_PARAM_STR(url) +	ZEND_PARSE_PARAMETERS_END();  	cp = curl_easy_init();  	if (!cp) { @@ -1988,7 +1988,7 @@ PHP_FUNCTION(curl_init)  	_php_curl_set_default_options(ch);  	if (url) { -		if (php_curl_option_url(ch, url, url_len) == FAILURE) { +		if (php_curl_option_url(ch, ZSTR_VAL(url), ZSTR_LEN(url)) == FAILURE) {  			_php_curl_close_ex(ch);  			RETURN_FALSE;  		} @@ -2080,9 +2080,9 @@ PHP_FUNCTION(curl_copy_handle)  	zval		*zid;  	php_curl	*ch, *dupch; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "r", &zid) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(1,1) +		Z_PARAM_RESOURCE(zid) +	ZEND_PARSE_PARAMETERS_END();  	if ((ch = (php_curl*)zend_fetch_resource(Z_RES_P(zid), le_curl_name, le_curl)) == NULL) {  		RETURN_FALSE; @@ -3311,9 +3311,9 @@ PHP_FUNCTION(curl_errno)  	zval		*zid;  	php_curl	*ch; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "r", &zid) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(1,1) +		Z_PARAM_RESOURCE(zid) +	ZEND_PARSE_PARAMETERS_END();  	if ((ch = (php_curl*)zend_fetch_resource(Z_RES_P(zid), le_curl_name, le_curl)) == NULL) {  		RETURN_FALSE; @@ -3534,24 +3534,25 @@ PHP_FUNCTION(curl_reset)     URL encodes the given string */  PHP_FUNCTION(curl_escape)  { -	char       *str = NULL, *res = NULL; -	size_t     str_len = 0; -	zval       *zid; -	php_curl   *ch; +	zend_string *str; +	char        *res; +	zval        *zid; +	php_curl    *ch; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "rs", &zid, &str, &str_len) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(2,2) +		Z_PARAM_RESOURCE(zid) +		Z_PARAM_STR(str) +	ZEND_PARSE_PARAMETERS_END();  	if ((ch = (php_curl*)zend_fetch_resource(Z_RES_P(zid), le_curl_name, le_curl)) == NULL) {  		RETURN_FALSE;  	} -	if (ZEND_SIZE_T_INT_OVFL(str_len)) { +	if (ZEND_SIZE_T_INT_OVFL(ZSTR_LEN(str))) {  		RETURN_FALSE;  	} -	if ((res = curl_easy_escape(ch->cp, str, str_len))) { +	if ((res = curl_easy_escape(ch->cp, ZSTR_VAL(str), ZSTR_LEN(str)))) {  		RETVAL_STRING(res);  		curl_free(res);  	} else { @@ -3564,25 +3565,26 @@ PHP_FUNCTION(curl_escape)     URL decodes the given string */  PHP_FUNCTION(curl_unescape)  { -	char       *str = NULL, *out = NULL; -	size_t     str_len = 0; -	int        out_len; -	zval       *zid; -	php_curl   *ch; +	char        *out = NULL; +	int          out_len; +	zval        *zid; +	zend_string *str; +	php_curl    *ch; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "rs", &zid, &str, &str_len) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(2,2) +		Z_PARAM_RESOURCE(zid) +		Z_PARAM_STR(str) +	ZEND_PARSE_PARAMETERS_END();  	if ((ch = (php_curl*)zend_fetch_resource(Z_RES_P(zid), le_curl_name, le_curl)) == NULL) {  		RETURN_FALSE;  	} -	if (ZEND_SIZE_T_INT_OVFL(str_len)) { +	if (ZEND_SIZE_T_INT_OVFL(ZSTR_LEN(str))) {  		RETURN_FALSE;  	} -	if ((out = curl_easy_unescape(ch->cp, str, str_len, &out_len))) { +	if ((out = curl_easy_unescape(ch->cp, ZSTR_VAL(str), ZSTR_LEN(str), &out_len))) {  		RETVAL_STRINGL(out, out_len);  		curl_free(out);  	} else { @@ -3601,9 +3603,10 @@ PHP_FUNCTION(curl_pause)  	zval       *zid;  	php_curl   *ch; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "rl", &zid, &bitmask) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(2,2) +		Z_PARAM_RESOURCE(zid) +		Z_PARAM_LONG(bitmask) +	ZEND_PARSE_PARAMETERS_END();  	if ((ch = (php_curl*)zend_fetch_resource(Z_RES_P(zid), le_curl_name, le_curl)) == NULL) {  		RETURN_FALSE; diff --git a/ext/curl/multi.c b/ext/curl/multi.c index c802edf46d..43202a980b 100644 --- a/ext/curl/multi.c +++ b/ext/curl/multi.c @@ -82,9 +82,10 @@ PHP_FUNCTION(curl_multi_add_handle)  	zval tmp_val;  	CURLMcode error = CURLM_OK; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "rr", &z_mh, &z_ch) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(2,2) +		Z_PARAM_RESOURCE(z_mh) +		Z_PARAM_RESOURCE(z_ch) +	ZEND_PARSE_PARAMETERS_END();  	if ((mh = (php_curlm *)zend_fetch_resource(Z_RES_P(z_mh), le_curl_multi_handle_name, le_curl_multi_handle)) == NULL) {  		RETURN_FALSE; @@ -169,9 +170,10 @@ PHP_FUNCTION(curl_multi_remove_handle)  	php_curl  *ch;  	CURLMcode error = CURLM_OK; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "rr", &z_mh, &z_ch) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(2,2) +		Z_PARAM_RESOURCE(z_mh) +		Z_PARAM_RESOURCE(z_ch) +	ZEND_PARSE_PARAMETERS_END();  	if ((mh = (php_curlm *)zend_fetch_resource(Z_RES_P(z_mh), le_curl_multi_handle_name, le_curl_multi_handle)) == NULL) {  		RETURN_FALSE; @@ -214,9 +216,11 @@ PHP_FUNCTION(curl_multi_select)  	struct timeval  to;  	CURLMcode error = CURLM_OK; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "r|d", &z_mh, &timeout) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(1,2) +		Z_PARAM_RESOURCE(z_mh) +		Z_PARAM_OPTIONAL +		Z_PARAM_DOUBLE(timeout) +	ZEND_PARSE_PARAMETERS_END();  	if ((mh = (php_curlm *)zend_fetch_resource(Z_RES_P(z_mh), le_curl_multi_handle_name, le_curl_multi_handle)) == NULL) {  		RETURN_FALSE; @@ -248,9 +252,10 @@ PHP_FUNCTION(curl_multi_exec)  	int        still_running;  	CURLMcode error = CURLM_OK; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "rz/", &z_mh, &z_still_running) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(2, 2) +		Z_PARAM_RESOURCE(z_mh) +		Z_PARAM_ZVAL_DEREF_EX(z_still_running, 0, 1) +	ZEND_PARSE_PARAMETERS_END();  	if ((mh = (php_curlm *)zend_fetch_resource(Z_RES_P(z_mh), le_curl_multi_handle_name, le_curl_multi_handle)) == NULL) {  		RETURN_FALSE; @@ -289,9 +294,9 @@ PHP_FUNCTION(curl_multi_getcontent)  	zval     *z_ch;  	php_curl *ch; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "r", &z_ch) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(1,1) +		Z_PARAM_RESOURCE(z_ch) +	ZEND_PARSE_PARAMETERS_END();  	if ((ch = (php_curl *)zend_fetch_resource(Z_RES_P(z_ch), le_curl_name, le_curl)) == NULL) {  		RETURN_FALSE; @@ -319,9 +324,11 @@ PHP_FUNCTION(curl_multi_info_read)  	int        queued_msgs;  	zval      *zmsgs_in_queue = NULL; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "r|z/", &z_mh, &zmsgs_in_queue) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(1, 2) +		Z_PARAM_RESOURCE(z_mh) +		Z_PARAM_OPTIONAL +		Z_PARAM_ZVAL_DEREF_EX(zmsgs_in_queue, 0, 1) +	ZEND_PARSE_PARAMETERS_END();  	if ((mh = (php_curlm *)zend_fetch_resource(Z_RES_P(z_mh), le_curl_multi_handle_name, le_curl_multi_handle)) == NULL) {  		RETURN_FALSE; @@ -369,9 +376,9 @@ PHP_FUNCTION(curl_multi_close)  	zval      *z_mh;  	php_curlm *mh; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "r", &z_mh) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(1,1) +		Z_PARAM_RESOURCE(z_mh) +	ZEND_PARSE_PARAMETERS_END();  	if ((mh = (php_curlm *)zend_fetch_resource(Z_RES_P(z_mh), le_curl_multi_handle_name, le_curl_multi_handle)) == NULL) {  		RETURN_FALSE; @@ -420,9 +427,9 @@ PHP_FUNCTION(curl_multi_errno)  	zval        *z_mh;  	php_curlm   *mh; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "r", &z_mh) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(1,1) +		Z_PARAM_RESOURCE(z_mh) +	ZEND_PARSE_PARAMETERS_END();  	if ((mh = (php_curlm *)zend_fetch_resource(Z_RES_P(z_mh), le_curl_multi_handle_name, le_curl_multi_handle)) == NULL) {  		RETURN_FALSE; @@ -440,9 +447,9 @@ PHP_FUNCTION(curl_multi_strerror)  	zend_long code;  	const char *str; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "l", &code) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(1,1) +		Z_PARAM_LONG(code) +	ZEND_PARSE_PARAMETERS_END();  	str = curl_multi_strerror(code);  	if (str) { @@ -598,9 +605,11 @@ PHP_FUNCTION(curl_multi_setopt)  	zend_long        options;  	php_curlm *mh; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "rlz", &z_mh, &options, &zvalue) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(3,3) +		Z_PARAM_RESOURCE(z_mh) +		Z_PARAM_LONG(options) +		Z_PARAM_ZVAL_DEREF(zvalue) +	ZEND_PARSE_PARAMETERS_END();  	if ((mh = (php_curlm *)zend_fetch_resource(Z_RES_P(z_mh), le_curl_multi_handle_name, le_curl_multi_handle)) == NULL) {  		RETURN_FALSE; diff --git a/ext/curl/share.c b/ext/curl/share.c index 983cc2750e..88a1f4e40e 100644 --- a/ext/curl/share.c +++ b/ext/curl/share.c @@ -59,9 +59,9 @@ PHP_FUNCTION(curl_share_close)  	zval *z_sh;  	php_curlsh *sh; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "r", &z_sh) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(1,1) +		Z_PARAM_RESOURCE(z_sh) +	ZEND_PARSE_PARAMETERS_END();  	if ((sh = (php_curlsh *)zend_fetch_resource(Z_RES_P(z_sh), le_curl_share_handle_name, le_curl_share_handle)) == NULL) {  		RETURN_FALSE; @@ -104,9 +104,11 @@ PHP_FUNCTION(curl_share_setopt)  	zend_long        options;  	php_curlsh *sh; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "rlz", &zid, &options, &zvalue) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(3,3) +		Z_PARAM_RESOURCE(zid) +		Z_PARAM_LONG(options) +		Z_PARAM_ZVAL_DEREF(zvalue) +	ZEND_PARSE_PARAMETERS_END();  	if ((sh = (php_curlsh *)zend_fetch_resource(Z_RES_P(zid), le_curl_share_handle_name, le_curl_share_handle)) == NULL) {  		RETURN_FALSE; @@ -138,9 +140,9 @@ PHP_FUNCTION(curl_share_errno)  	zval        *z_sh;  	php_curlsh  *sh; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "r", &z_sh) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(1,1) +		Z_PARAM_RESOURCE(z_sh) +	ZEND_PARSE_PARAMETERS_END();  	if ((sh = (php_curlsh *)zend_fetch_resource(Z_RES_P(z_sh), le_curl_share_handle_name, le_curl_share_handle)) == NULL) {  		RETURN_FALSE; @@ -159,9 +161,9 @@ PHP_FUNCTION(curl_share_strerror)  	zend_long code;  	const char *str; -	if (zend_parse_parameters(ZEND_NUM_ARGS(), "l", &code) == FAILURE) { -		return; -	} +	ZEND_PARSE_PARAMETERS_START(1,1) +		Z_PARAM_LONG(code) +	ZEND_PARSE_PARAMETERS_END();  	str = curl_share_strerror(code);  	if (str) { | 
