From: Zeev Suraski <zeev@php.net>
Date: Wed, 27 Jun 2001 15:47:31 +0000 (+0000)
Subject: Fix leak in the patch, and revert a couple of lines I didn't mean to commit
X-Git-Tag: PRE_TSRM_MERGE_PATCH~323
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=2536259557a4986f0af4fdaa129898bdadfadcf4;p=php

Fix leak in the patch, and revert a couple of lines I didn't mean to commit
---

diff --git a/Zend/zend_execute.c b/Zend/zend_execute.c
index 667a6d1689..c6a71264c9 100644
--- a/Zend/zend_execute.c
+++ b/Zend/zend_execute.c
@@ -328,6 +328,7 @@ static inline void zend_assign_to_variable(znode *result, znode *op1, znode *op2
 							int i;
 
 							if (T->EA.data.str_offset.str->value.str.len==0) {
+								STR_FREE(T->EA.data.str_offset.str->value.str.val);
 								T->EA.data.str_offset.str->value.str.val = (char *) emalloc(T->EA.data.str_offset.offset+1+1);
 							} else {
 								T->EA.data.str_offset.str->value.str.val = (char *) erealloc(T->EA.data.str_offset.str->value.str.val, T->EA.data.str_offset.offset+1+1);
@@ -1085,12 +1086,9 @@ ZEND_API void execute(zend_op_array *op_array ELS_DC)
 				EG(binary_op) = boolean_xor_function;
 		  	    /* Fall through */
 binary_op_addr:
-				{
-					zval *zp1 = get_zval_ptr(&opline->op1, Ts, &EG(free_op1), BP_VAR_R);
-					zval *zp2 = get_zval_ptr(&opline->op2, Ts, &EG(free_op1), BP_VAR_R);
-
-					EG(binary_op)(&Ts[opline->result.u.var].tmp_var, zp1, zp2);
-				}
+				EG(binary_op)(&Ts[opline->result.u.var].tmp_var, 
+					get_zval_ptr(&opline->op1, Ts, &EG(free_op1), BP_VAR_R),
+					get_zval_ptr(&opline->op2, Ts, &EG(free_op2), BP_VAR_R) );
 				FREE_OP(&opline->op1, EG(free_op1));
 				FREE_OP(&opline->op2, EG(free_op2));
 				NEXT_OPCODE();