array_map: Delay allocation of array_pos

ndossche · ndossche · commit 9bf1240d9f68 · 2025-11-08T23:29:24.000+01:00
diff --git a/ext/standard/array.c b/ext/standard/array.c
@@ -6854,19 +6854,17 @@ PHP_FUNCTION(array_map)
 			} ZEND_HASH_FOREACH_END();
 		}
 	} else {
-		uint32_t *array_pos = (HashPosition *)ecalloc(n_arrays, sizeof(HashPosition));
-
 		for (i = 0; i < n_arrays; i++) {
 			if (Z_TYPE(arrays[i]) != IS_ARRAY) {
 				zend_argument_type_error(i + 2, "must be of type array, %s given", zend_zval_value_name(&arrays[i]));
-				efree(array_pos);
 				RETURN_THROWS();
 			}
 			if (zend_hash_num_elements(Z_ARRVAL(arrays[i])) > maxlen) {
 				maxlen = zend_hash_num_elements(Z_ARRVAL(arrays[i]));
 			}
 		}
 
+		uint32_t *array_pos = ecalloc(n_arrays, sizeof(HashPosition));
 		array_init_size(return_value, maxlen);
 
 		if (!ZEND_FCI_INITIALIZED(fci)) {

Original file line number	Diff line number	Diff line change
`@@ -6854,19 +6854,17 @@ PHP_FUNCTION(array_map)`
`6854`	`6854`	`} ZEND_HASH_FOREACH_END();`
`6855`	`6855`	`}`
`6856`	`6856`	`} else {`
`6857`		`- uint32_t array_pos = (HashPosition )ecalloc(n_arrays, sizeof(HashPosition));`
`6858`		`-`
`6859`	`6857`	`for (i = 0; i < n_arrays; i++) {`
`6860`	`6858`	`if (Z_TYPE(arrays[i]) != IS_ARRAY) {`
`6861`	`6859`	`zend_argument_type_error(i + 2, "must be of type array, %s given", zend_zval_value_name(&arrays[i]));`
`6862`		`- efree(array_pos);`
`6863`	`6860`	`RETURN_THROWS();`
`6864`	`6861`	`}`
`6865`	`6862`	`if (zend_hash_num_elements(Z_ARRVAL(arrays[i])) > maxlen) {`
`6866`	`6863`	`maxlen = zend_hash_num_elements(Z_ARRVAL(arrays[i]));`
`6867`	`6864`	`}`
`6868`	`6865`	`}`
`6869`	`6866`
	`6867`	`+ uint32_t *array_pos = ecalloc(n_arrays, sizeof(HashPosition));`
`6870`	`6868`	`array_init_size(return_value, maxlen);`
`6871`	`6869`
`6872`	`6870`	`if (!ZEND_FCI_INITIALIZED(fci)) {`