diff options
| author | Xinchen Hui <laruence@php.net> | 2012-02-12 05:32:24 +0000 |
|---|---|---|
| committer | Xinchen Hui <laruence@php.net> | 2012-02-12 05:32:24 +0000 |
| commit | 33125e2312045e26853c5a44f255d56b9742edaf (patch) | |
| tree | 33c76cbe0637873e5247b4a43591667f3907693e /ext/standard/array.c | |
| parent | c578917e306b452493fed4a0aa2ecbd3a8c3f252 (diff) | |
| download | php-git-33125e2312045e26853c5a44f255d56b9742edaf.tar.gz | |
Improved fix for #61058, and add test script
Diffstat (limited to 'ext/standard/array.c')
| -rw-r--r-- | ext/standard/array.c | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/ext/standard/array.c b/ext/standard/array.c index 3ca1a69a5e..f18d558bbe 100644 --- a/ext/standard/array.c +++ b/ext/standard/array.c @@ -1557,15 +1557,16 @@ PHP_FUNCTION(array_fill) array_init_size(return_value, num); num--; - zval_add_ref(&val); - if (zend_hash_index_update(Z_ARRVAL_P(return_value), start_key, &val, sizeof(zval *), NULL) == FAILURE) { - zval_ptr_dtor(&val); - } + zend_hash_index_update(Z_ARRVAL_P(return_value), start_key, &val, sizeof(zval *), NULL); + zval_add_ref(&val); while (num--) { - zval_add_ref(&val); - if (zend_hash_next_index_insert(Z_ARRVAL_P(return_value), &val, sizeof(zval *), NULL) == FAILURE) { - zval_ptr_dtor(&val); + if (zend_hash_next_index_insert(Z_ARRVAL_P(return_value), &val, sizeof(zval *), NULL) == SUCCESS) { + zval_add_ref(&val); + } else { + zval_dtor(return_value); + php_error_docref(NULL TSRMLS_CC, E_WARNING, "Cannot add element to the array as the next element is already occupied"); + RETURN_FALSE; } } } |