Compilation warning

[GoogleCodeExporter]

bin/sh /root/DEBS/PHP-FPM/5.2.17/php-5.2.17/libtool --silent 
--preserve-dup-deps --mode=compile gcc  -Iext/libxml/ 
-I/root/DEBS/PHP-FPM/5.2.17
/php-5.2.17/ext/libxml/ -DPHP_ATOM_INC 
-I/root/DEBS/PHP-FPM/5.2.17/php-5.2.17/include 
-I/root/DEBS/PHP-FPM/5.2.17/php-5.2.17/main -I/root/DEBS/
PHP-FPM/5.2.17/php-5.2.17 -I/usr/include/libxml2 
-I/root/DEBS/PHP-FPM/5.2.17/php-5.2.17/ext/date/lib -I/usr/include/freetype2 
-I/root/DEBS/PHP-
FPM/5.2.17/php-5.2.17/ext/mbstring/oniguruma 
-I/root/DEBS/PHP-FPM/5.2.17/php-5.2.17/ext/mbstring/libmbfl 
-I/root/DEBS/PHP-FPM/5.2.17/php-5.2.17
/ext/mbstring/libmbfl/mbfl -I/usr/include/mysql 
-I/root/DEBS/PHP-FPM/5.2.17/php-5.2.17/TSRM 
-I/root/DEBS/PHP-FPM/5.2.17/php-5.2.17/Zend    -I/u
sr/include -g -O2  -c /root/DEBS/PHP-FPM/5.2.17/php-5.2.17/ext/libxml/libxml.c 
-o ext/libxml/libxml.lo 
/root/DEBS/PHP-FPM/5.2.17/php-5.2.17/ext/libxml/libxml.c: In function 
‘_php_libxml_destroy_fci’:
/root/DEBS/PHP-FPM/5.2.17/php-5.2.17/ext/libxml/libxml.c:279:4: warning: 
passing argument 1 of ‘_zval_ptr_dtor’ from incompatible pointer type 
[enabled by default]
In file included from /root/DEBS/PHP-FPM/5.2.17/php-5.2.17/Zend/zend.h:682:0,
                 from /root/DEBS/PHP-FPM/5.2.17/php-5.2.17/main/php.h:34,
                 from /root/DEBS/PHP-FPM/5.2.17/php-5.2.17/ext/libxml/libxml.c:28:
/root/DEBS/PHP-FPM/5.2.17/php-5.2.17/Zend/zend_variables.h:50:15: note: 
expected ‘struct zval **’ but argument is of type ‘struct zval ***’

In libxml.c:
static void _php_libxml_destroy_fci(zend_fcall_info *fci)
{
        if (fci->size > 0) {
                zval_ptr_dtor(&fci->function_name);
                if (fci->object_pp != NULL) {
                        zval_ptr_dtor(&fci->object_pp);
                                     ^^^^^^^^^^^^^^^^^
Shouldn't it be:
                        zval_ptor_dtor(fci->object_pp); ?

                }
                fci->size = 0;
        }
}

Original issue reported on code.google.com by gglate...@gmail.com on 6 May 2013 at 10:44

[GoogleCodeExporter]

I've experienced the same issue. Passing the pointer address is wrong. Passing 
the pointer itself, seems to have worked fine for me.

Original comment by jijwilho...@gmail.com on 15 Jul 2013 at 11:35

[GoogleCodeExporter]

Commited, thanks

Original comment by svyatosl...@gmail.com on 17 Jul 2013 at 7:16

• Changed state: Fixed