- Another small fix.

Andi Gutmans · Andi Gutmans · commit 419c68f9ecaf · 1999-12-11T12:17:21.000Z
diff --git a/Zend/zend_execute.c b/Zend/zend_execute.c
@@ -1549,16 +1549,18 @@ binary_assign_op_addr: {
 					
 					if (opline->extended_value) {
 						retval_ptr_ptr = get_zval_ptr_ptr(&opline->op1, Ts, BP_VAR_R);
-						SEPARATE_ZVAL(retval_ptr_ptr);
-						(*retval_ptr_ptr)->is_ref = 1;
+
+						if (!PZVAL_IS_REF(*retval_ptr_ptr)) {
+							SEPARATE_ZVAL(retval_ptr_ptr);
+							(*retval_ptr_ptr)->is_ref = 1;
+						}
 						(*retval_ptr_ptr)->refcount++;
 						efree(*EG(return_value_ptr_ptr));
 						(*EG(return_value_ptr_ptr)) = (*retval_ptr_ptr);
 					} else {
 						retval_ptr = get_zval_ptr(&opline->op1, Ts, &EG(free_op1), BP_VAR_R);
 					
 						if (!EG(free_op1)) { /* Not a temp var */
-							
 							if (PZVAL_IS_REF(retval_ptr) && retval_ptr->refcount > 0) {
 								**EG(return_value_ptr_ptr) = *retval_ptr;
 								(*EG(return_value_ptr_ptr))->is_ref = 0;
