Commit 53df3ae

committed

ext/curl: Use Z_ARRVAL_P instead of HASH_OF

We already check in advance that it is an array

1 parent 23b8d64 commit 53df3aeCopy full SHA for 53df3ae

File tree

-2

lines changed

-2

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1540,7 +1540,7 @@ static inline zend_result build_mime_structure_from_hash(php_curl ch, zval zpo`
`1540`	`1540`	`if (Z_TYPE_P(current) == IS_ARRAY) {`
`1541`	`1541`	`zval *current_element;`
`1542`	`1542`
`1543`		`- ZEND_HASH_FOREACH_VAL(HASH_OF(current), current_element) {`
	`1543`	`+ ZEND_HASH_FOREACH_VAL(Z_ARRVAL_P(current), current_element) {`
`1544`	`1544`	`add_simple_field(mime, string_key, current_element);`
`1545`	`1545`	`} ZEND_HASH_FOREACH_END();`
`1546`	`1546`
`@@ -2186,7 +2186,7 @@ static zend_result _php_curl_setopt(php_curl ch, zend_long option, zval zvalue`
`2186`	`2186`
`2187`	`2187`	`case CURLOPT_POSTFIELDS:`
`2188`	`2188`	`if (Z_TYPE_P(zvalue) == IS_ARRAY) {`
`2189`		`- if (zend_hash_num_elements(HASH_OF(zvalue)) == 0) {`
	`2189`	`+ if (zend_hash_num_elements(Z_ARRVAL_P(zvalue)) == 0) {`
`2190`	`2190`	`/* no need to build the mime structure for empty hashtables;`
`2191`	`2191`	`also works around https://github.com/curl/curl/issues/6455 */`
`2192`	`2192`	`curl_easy_setopt(ch->cp, CURLOPT_POSTFIELDS, "");`

Comments

(0)