array_map: Delay allocation of array_pos

ndossche · ndossche · commit dbce0334be40 · 2025-11-08T18:10:36.000+01:00
diff --git a/ext/standard/array.c b/ext/standard/array.c
@@ -6853,19 +6853,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
`@@ -6853,19 +6853,17 @@ PHP_FUNCTION(array_map)`
`6853`	`6853`	`} ZEND_HASH_FOREACH_END();`
`6854`	`6854`	`}`
`6855`	`6855`	`} else {`
`6856`		`- uint32_t array_pos = (HashPosition )ecalloc(n_arrays, sizeof(HashPosition));`
`6857`		`-`
`6858`	`6856`	`for (i = 0; i < n_arrays; i++) {`
`6859`	`6857`	`if (Z_TYPE(arrays[i]) != IS_ARRAY) {`
`6860`	`6858`	`zend_argument_type_error(i + 2, "must be of type array, %s given", zend_zval_value_name(&arrays[i]));`
`6861`		`- efree(array_pos);`
`6862`	`6859`	`RETURN_THROWS();`
`6863`	`6860`	`}`
`6864`	`6861`	`if (zend_hash_num_elements(Z_ARRVAL(arrays[i])) > maxlen) {`
`6865`	`6862`	`maxlen = zend_hash_num_elements(Z_ARRVAL(arrays[i]));`
`6866`	`6863`	`}`
`6867`	`6864`	`}`
`6868`	`6865`
	`6866`	`+ uint32_t *array_pos = ecalloc(n_arrays, sizeof(HashPosition));`
`6869`	`6867`	`array_init_size(return_value, maxlen);`
`6870`	`6868`
`6871`	`6869`	`if (!ZEND_FCI_INITIALIZED(fci)) {`