Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:2903
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Message-ID: <1056737912.3efc8a7881a32@webmail.purdue.edu>
Date: Fri, 27 Jun 2003 13:18:32 -0500
To: internals@lists.php.net
MIME-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit
User-Agent: Internet Messaging Program (IMP) 3.2-cvs
Subject: [PATCH] extend zend_parse_parameters() functionality (2nd attempt)
From: fuhs@purdue.edu

Hello,

I propose that a new type, 'Z', be added in order to allow the extension coders
access to the zval** which was available with the now deprecated
zend_get_parameters().

I came to this conclusion after tracing some segfaults to a section of code
similar to this:

    zval *zend_value;
    if(zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z", &zend_value) ==
       FAILURE)
        return;

    convert_to_array_ex(&zend_value);

I realized after sifting through the Zend code that the proper handling of zval*
types is just as important as that of the zval's. This modification will
give extension programmers access to legitimate zval*'s and their corresponding
zval.

Zend/zend_API.c RELEASE ver 4.3.1
425c425,434
<
---
>               case 'Z':
>                       {
>                               zval ***p=va_arg(*va, zval ***);
>                               if(Z_TYPE_PP(arg) == IS_NULL && return_null){
>                                      *p = NULL;
>                               } else {
>                                      *p = arg;
>                               }
>                       }
>                       break;
474c483
<                       case 'z':
---
>                       case 'z': case 'Z':


Josh