For Programmers: Free Programming Magazines  


Home > Archive > PHP Zend Engine > July 2004 > Re: [ZEND-ENGINE-CVS] cvs: ZendEngine2 / zend_execute_API.c









You are viewing an archived Text-only version of the thread. To view this thread in it's original format and/or if you want to reply to this thread please [click here]

 

Author Re: [ZEND-ENGINE-CVS] cvs: ZendEngine2 / zend_execute_API.c
Wez Furlong

2004-07-19, 8:55 am

Yeah, that's the one I was hinting at.

--Wez.

On Mon, 19 Jul 2004 11:41:24 +0400, Antony Dovgal <tony2001@phpclub.net> wrote:
> On Sun, 18 Jul 2004 18:51:20 +0100
> Wez Furlong <wez@thebrainroom.com> wrote:
>
>
> Wez, it seems, that there is the same problem in zend_execute.c. See the
> diff:
>
> Index: zend_execute.c
> ========================================
===========================
> RCS file: /repository/ZendEngine2/zend_execute.c,v
> retrieving revision 1.654
> diff -u -r1.654 zend_execute.c
> --- zend_execute.c 19 Jul 2004 07:19:01 -0000 1.654
> +++ zend_execute.c 19 Jul 2004 07:36:01 -0000
> @@ -1330,7 +1330,9 @@
> }
>
> #define RETURN_FROM_EXECUTE_LOOP(execute_data)
> \- efree(EX(Ts));
>
> \+ if (EX(Ts)) {
> \+
> efree(EX(Ts));
> \+ }
>
> \ EG(in_execution) = EX(original_in_execution);
> \
> EG(current_execute_data) = EX(prev_execute_data);
> \ return 1; /* CHECK_ME */
> @@ -1349,7 +1351,10 @@
> /* Initialize execute_data */
> EX(fbc) = NULL;
> EX(object) = NULL;
> - EX(Ts) = (temp_variable *) safe_emalloc(sizeof(temp_variable),
> op_array->T, 0);+ EX(Ts) = NULL;
> + if (op_array->T) {
> + EX(Ts) = (temp_variable *)
> safe_emalloc(sizeof(temp_variable), op_array->T, 0);+ }
> EX(op_array) = op_array;
> EX(original_in_execution) = EG(in_execution);
> EX(prev_execute_data) = EG(current_execute_data);
>
> Here you can find valgrind's output:
> http://lists.php.net/php.internals/11162
>
> ---
> WBR,
> Antony Dovgal aka tony2001
> tony2001@phpclub.net || antony@dovgal.com
>
>
>
> --
> Zend Engine CVS Mailing List (http://cvs.php.net/)
> To unsubscribe, visit: http://www.php.net/unsub.php
>
>

Sponsored Links







Also available: Server administration forum archive | Web Design forum archive | Software forum archive | Hardware reviews archive

Copyright 2008 codecomments.com