php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #66218 zend_register_functions breaks reflection
Submitted: 2013-12-02 13:34 UTC Modified: 2013-12-02 16:02 UTC
From: remi@php.net Assigned: remi
Status: Closed Package: Reflection related
PHP Version: 5.5.6 OS: irrevelant
Private report: No CVE-ID:
 [2013-12-02 13:34 UTC] remi@php.net
Description:
------------
Functions registered using zend_register_functions instead of zend_module_entry.functions are not seen on reflection.

Ex: additional_functions from api_module_entry.

Ex: in CLI, dl, cli_set_process_title and cli_get_process_title

Note: also affects functions overrided in extension (should be be reported in extension, where overrided, not in original extension)


Test script:
---------------
Affected test
- php --re standard
- get_extension_funcs("standard"));
- $r=new ReflectionExtension('standard'); print_r($r->getFunctions());


Expected result:
----------------
dl, cli_set_process_title and cli_get_process_title in list

Actual result:
--------------
dl, cli_set_process_title and cli_get_process_title missing

Patches

reflection.patch (last revision 2013-12-02 16:02 UTC) by remi@php.net)
ReflectionExtension_bug66218.phpt (last revision 2013-12-02 14:41 UTC) by remi@php.net)
bug66218.phpt (last revision 2013-12-02 13:42 UTC) by remi@php.net)

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2013-12-02 13:42 UTC] remi@php.net
The following patch has been added/updated:

Patch Name: bug66218.phpt
Revision:   1385991746
URL:        https://bugs.php.net/patch-display.php?bug=66218&patch=bug66218.phpt&revision=1385991746
 [2013-12-02 14:13 UTC] remi@php.net
The following patch has been added/updated:

Patch Name: reflection.patch
Revision:   1385993630
URL:        https://bugs.php.net/patch-display.php?bug=66218&patch=reflection.patch&revision=1385993630
 [2013-12-02 14:41 UTC] remi@php.net
The following patch has been added/updated:

Patch Name: ReflectionExtension_bug66218.phpt
Revision:   1385995303
URL:        https://bugs.php.net/patch-display.php?bug=66218&patch=ReflectionExtension_bug66218.phpt&revision=1385995303
 [2013-12-02 16:02 UTC] remi@php.net
The following patch has been added/updated:

Patch Name: reflection.patch
Revision:   1386000124
URL:        https://bugs.php.net/patch-display.php?bug=66218&patch=reflection.patch&revision=1386000124
 [2013-12-02 16:02 UTC] remi@php.net
-Summary: zend_register_functions issue break reflection +Summary: zend_register_functions breaks reflection -Status: Open +Status: Assigned -Assigned To: +Assigned To: remi
 [2013-12-02 16:02 UTC] remi@php.net
Last patch include the 2 new tests.
 [2013-12-10 15:11 UTC] remi@php.net
Automatic comment on behalf of remi
Revision: http://git.php.net/?p=php-src.git;a=commit;h=3e963f8eb44863ef3d758eabe791190b0fd7bb9a
Log: Fixed Bug #66218 zend_register_functions breaks reflection
 [2013-12-10 15:11 UTC] remi@php.net
-Status: Assigned +Status: Closed
 [2013-12-10 18:47 UTC] ab@php.net
Automatic comment on behalf of remi
Revision: http://git.php.net/?p=php-src.git;a=commit;h=3e963f8eb44863ef3d758eabe791190b0fd7bb9a
Log: Fixed Bug #66218 zend_register_functions breaks reflection
 
PHP Copyright © 2001-2014 The PHP Group
All rights reserved.
Last updated: Thu Apr 17 21:01:56 2014 UTC