diff options
| author | Zeev Suraski <zeev@php.net> | 2004-02-08 17:23:20 +0000 |
|---|---|---|
| committer | Zeev Suraski <zeev@php.net> | 2004-02-08 17:23:20 +0000 |
| commit | 473927ece8baed35ccffe3895a90caf1d6652b53 (patch) | |
| tree | c4c3917c8e15ab632f8056c9c0d4141889c1c1b3 /Zend/zend_execute.c | |
| parent | 8a129bfc47db65ad14fe947e77c1f52ac922868c (diff) | |
| download | php-git-473927ece8baed35ccffe3895a90caf1d6652b53.tar.gz | |
Fix write-mode of overloaded objects when using array dimensions
Diffstat (limited to 'Zend/zend_execute.c')
| -rw-r--r-- | Zend/zend_execute.c | 34 |
1 files changed, 16 insertions, 18 deletions
diff --git a/Zend/zend_execute.c b/Zend/zend_execute.c index 5e8209e3d2..6117695549 100644 --- a/Zend/zend_execute.c +++ b/Zend/zend_execute.c @@ -911,26 +911,24 @@ static void zend_fetch_dimension_address(znode *result, znode *op1, znode *op2, } break; case IS_OBJECT: - if (type == BP_VAR_R || type == BP_VAR_RW) { - if (!Z_OBJ_HT_P(container)->read_dimension) { - zend_error(E_ERROR, "Cannot use object as array"); - } else { - zval *dim = get_zval_ptr(op2, Ts, &EG(free_op2), BP_VAR_R); - zval *overloaded_result = Z_OBJ_HT_P(container)->read_dimension(container, dim TSRMLS_CC); + if (!Z_OBJ_HT_P(container)->read_dimension) { + zend_error(E_ERROR, "Cannot use object as array"); + } else { + zval *dim = get_zval_ptr(op2, Ts, &EG(free_op2), BP_VAR_R); + zval *overloaded_result = Z_OBJ_HT_P(container)->read_dimension(container, dim, type TSRMLS_CC); - if (overloaded_result) { - if (type == BP_VAR_RW && !overloaded_result->is_ref) { - zend_error(E_ERROR, "Objects used as arrays in post/pre increment/decrement must return values by reference"); - } - - *retval = &overloaded_result; - } else { - *retval = &EG(error_zval_ptr); + if (overloaded_result) { + if (type == BP_VAR_RW && !overloaded_result->is_ref) { + zend_error(E_ERROR, "Objects used as arrays in post/pre increment/decrement must return values by reference"); } - AI_USE_PTR(T(result->u.var).var); - FREE_OP(Ts, op2, EG(free_op2)); - SELECTIVE_PZVAL_LOCK(**retval, result); + + *retval = &overloaded_result; + } else { + *retval = &EG(error_zval_ptr); } + AI_USE_PTR(T(result->u.var).var); + FREE_OP(Ts, op2, EG(free_op2)); + SELECTIVE_PZVAL_LOCK(**retval, result); } break; default: { @@ -1559,7 +1557,7 @@ static inline int zend_binary_assign_op_obj_helper(int (*binary_op)(zval *result z = Z_OBJ_HT_P(object)->read_property(object, property, 0 TSRMLS_CC); break; case ZEND_ASSIGN_DIM: - z = Z_OBJ_HT_P(object)->read_dimension(object, property TSRMLS_CC); + z = Z_OBJ_HT_P(object)->read_dimension(object, property, BP_VAR_W TSRMLS_CC); break; } SEPARATE_ZVAL_IF_NOT_REF(&z); |