Skip to content

Commit e75804c

Browse files
committed
Merge branch 'PHP-5.4' into PHP-5.5
2 parents 59ec22b + d24ac69 commit e75804c

File tree

2 files changed

+44
-10
lines changed

2 files changed

+44
-10
lines changed

ext/spl/spl_fixedarray.c

+15-10
Original file line numberDiff line numberDiff line change
@@ -659,22 +659,27 @@ SPL_METHOD(SplFixedArray, count)
659659
*/
660660
SPL_METHOD(SplFixedArray, toArray)
661661
{
662-
zval *ret, *tmp;
663-
HashTable *ret_ht, *obj_ht;
662+
spl_fixedarray_object *intern;
664663

665664
if (FAILURE == zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "")) {
666665
return;
667666
}
668667

669-
ALLOC_HASHTABLE(ret_ht);
670-
zend_hash_init(ret_ht, 0, NULL, ZVAL_PTR_DTOR, 0);
671-
ALLOC_INIT_ZVAL(ret);
672-
Z_TYPE_P(ret) = IS_ARRAY;
673-
obj_ht = spl_fixedarray_object_get_properties(getThis() TSRMLS_CC);
674-
zend_hash_copy(ret_ht, obj_ht, (copy_ctor_func_t) zval_add_ref, (void *) &tmp, sizeof(zval *));
675-
Z_ARRVAL_P(ret) = ret_ht;
668+
intern = (spl_fixedarray_object *)zend_object_store_get_object(getThis() TSRMLS_CC);
676669

677-
RETURN_ZVAL(ret, 1, 1);
670+
array_init(return_value);
671+
if (intern->array) {
672+
int i = 0;
673+
for (; i < intern->array->size; i++) {
674+
if (intern->array->elements[i]) {
675+
zend_hash_index_update(Z_ARRVAL_P(return_value), i, (void *)&intern->array->elements[i], sizeof(zval *), NULL);
676+
Z_ADDREF_P(intern->array->elements[i]);
677+
} else {
678+
zend_hash_index_update(Z_ARRVAL_P(return_value), i, (void *)&EG(uninitialized_zval_ptr), sizeof(zval *), NULL);
679+
Z_ADDREF_P(EG(uninitialized_zval_ptr));
680+
}
681+
}
682+
}
678683
}
679684
/* }}} */
680685

ext/spl/tests/bug64264.phpt

+29
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
--TEST--
2+
Bug #64264 (SPLFixedArray toArray problem)
3+
--FILE--
4+
<?php
5+
class MyFixedArray extends \SplFixedArray {
6+
protected $foo;
7+
protected $bar;
8+
}
9+
10+
$myFixedArr = new MyFixedArray(1);
11+
$myFixedArr[0] = 'foo';
12+
$myFixedArr->setSize(2);
13+
$myFixedArr[1] = 'bar';
14+
$myFixedArr->setSize(5);
15+
$array = $myFixedArr->toArray();
16+
$array[2] = "ERROR";
17+
$array[3] = "ERROR";
18+
$array[4] = "ERROR";
19+
unset($array[4]);
20+
$myFixedArr->setSize(2);
21+
22+
print_r($myFixedArr->toArray());
23+
?>
24+
--EXPECTF--
25+
Array
26+
(
27+
[0] => foo
28+
[1] => bar
29+
)

0 commit comments

Comments
 (0)