cvs: ZendEngine2 / zend_execute.c zend_hash.c

stas Thu Aug 1 12:07:19 2002 EDT Modified files: /ZendEngine2 zend_execute.c zend_hash.c Log: MFZE1 Index: ZendEngine2/zend_execute.c diff -u ZendEngine2/zend_execute.c:1.356 ZendEngine2/zend_execute.c:1.357 --- ZendEngine2/zend_execute.c:1.356 Tue Jul 30 18:19:50 2002 +++ ZendEngine2/zend_execute.c Thu Aug 1 12:07:19 2002 @@ -791,7 +791,11 @@ zval *new_zval = &EG(uninitialized_zval); new_zval->refcount++; - zend_hash_next_index_insert(container->value.ht, &new_zval, sizeof(zval *), (void **) retval); + if (zend_hash_next_index_insert(container->value.ht, &new_zval, sizeof(zval *), (void **) retval) == FAILURE) { + zend_error(E_WARNING, "Cannot add element to the array as the nexxt element is already occupied"); + *retval = &EG(uninitialized_zval_ptr); + new_zval->refcount--; + } } else { *retval = zend_fetch_dimension_address_inner(container->value.ht, op2, Ts, type TSRMLS_CC); } Index: ZendEngine2/zend_hash.c diff -u ZendEngine2/zend_hash.c:1.87 ZendEngine2/zend_hash.c:1.88 --- ZendEngine2/zend_hash.c:1.87 Tue Jul 23 15:29:02 2002 +++ ZendEngine2/zend_hash.c Thu Aug 1 12:07:19 2002 @@ -393,7 +393,7 @@ } UPDATE_DATA(ht, p, pData, nDataSize); HANDLE_UNBLOCK_INTERRUPTIONS(); - if (h >= ht->nNextFreeElement) { + if ((long)h >= (long)ht->nNextFreeElement) { ht->nNextFreeElement = h + 1; } if (pDest) { @@ -421,7 +421,7 @@ CONNECT_TO_GLOBAL_DLLIST(p, ht); HANDLE_UNBLOCK_INTERRUPTIONS(); - if (h >= ht->nNextFreeElement) { + if ((long)h >= (long)ht->nNextFreeElement) { ht->nNextFreeElement = h + 1; } ht->nNumOfElements++;