# 5.3 cvs zend_do_begin_dynamic_function_call ZEND_OP_DATA op2 not initialized

- Thread: https://externals.io/message/42218
- Original archive: https://news-web.php.net/php.internals/42218
- Started by Xuefer on 2008-12-12T04:58:19+00:00
- Messages: 2
- Reply by email to internals@lists.php.net
- More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt

---

## #42218 — Xuefer — 2008-12-12T04:58:19+00:00

URL: https://externals.io/message/42218#42218

opline2->op2.op_type == 0, which should be opline2->op2.op_type ==
IS_UNUSED. patch followed:

$ cvs diff zend_compile.c
Index: zend_compile.c
===================================================================
RCS file: /repository/ZendEngine2/zend_compile.c,v
retrieving revision 1.647.2.27.2.41.2.95
diff -u -r1.647.2.27.2.41.2.95 zend_compile.c
--- zend_compile.c  26 Nov 2008 00:57:05 -0000  1.647.2.27.2.41.2.95
+++ zend_compile.c  12 Dec 2008 04:54:42 -0000
@@ -1664,6 +1664,7 @@
        Z_LVAL(opline2->op1.u.constant) = prefix_len;
        /* this is the hash of the non-prefixed part, lowercased */
        opline2->extended_value = zend_hash_func(slash+1, name_len+1);
+       SET_UNUSED(opline2->op2);
    } else {
        opline->opcode = ZEND_INIT_FCALL_BY_NAME;
        opline->op2 = *function_name;

---

## #42265 — Stanislav Malyshev — 2008-12-15T11:32:09+00:00

URL: https://externals.io/message/42218#42265 · In reply to: #42218

Hi!

Thanks, fixed.

Xuefer wrote:
> opline2->op2.op_type == 0, which should be opline2->op2.op_type ==
> IS_UNUSED. patch followed:
> 
> $ cvs diff zend_compile.c
> Index: zend_compile.c
> ===================================================================
> RCS file: /repository/ZendEngine2/zend_compile.c,v
> retrieving revision 1.647.2.27.2.41.2.95
> diff -u -r1.647.2.27.2.41.2.95 zend_compile.c
> --- zend_compile.c  26 Nov 2008 00:57:05 -0000  1.647.2.27.2.41.2.95
> +++ zend_compile.c  12 Dec 2008 04:54:42 -0000
> @@ -1664,6 +1664,7 @@
>         Z_LVAL(opline2->op1.u.constant) = prefix_len;
>         /* this is the hash of the non-prefixed part, lowercased */
>         opline2->extended_value = zend_hash_func(slash+1, name_len+1);
> +       SET_UNUSED(opline2->op2);
>     } else {
>         opline->opcode = ZEND_INIT_FCALL_BY_NAME;
>         opline->op2 = *function_name;
> 

-- 
Stanislav Malyshev, Zend Software Architect
stas@zend.com   http://www.zend.com/
(408)253-8829   MSN: stas@zend.com