From aa394e70ff8cda63ee4382d2488d80936e5122dc Mon Sep 17 00:00:00 2001 From: Nikita Popov Date: Fri, 19 Dec 2014 21:40:54 +0100 Subject: Fix bug #67111 Loop variables need to be freed for both "break" and "continue". I'm adding the test to Zend/ because it's good to have a test for this even without opcache. --- ext/opcache/Optimizer/pass2.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'ext/opcache/Optimizer/pass2.c') diff --git a/ext/opcache/Optimizer/pass2.c b/ext/opcache/Optimizer/pass2.c index 30708a0935..8704b787a9 100644 --- a/ext/opcache/Optimizer/pass2.c +++ b/ext/opcache/Optimizer/pass2.c @@ -175,9 +175,9 @@ if (ZEND_OPTIMIZER_PASS_2 & OPTIMIZATION_LEVEL) { jmp_to = &op_array->brk_cont_array[array_offset]; array_offset = jmp_to->parent; if (--nest_levels > 0) { - if (opline->opcode == ZEND_BRK && - (op_array->opcodes[jmp_to->brk].opcode == ZEND_FREE || - op_array->opcodes[jmp_to->brk].opcode == ZEND_SWITCH_FREE)) { + if (op_array->opcodes[jmp_to->brk].opcode == ZEND_FREE || + op_array->opcodes[jmp_to->brk].opcode == ZEND_SWITCH_FREE + ) { dont_optimize = 1; break; } -- cgit v1.2.1