Edit report at https://bugs.php.net/bug.php?id=63159&edit=1
ID: 63159
Comment by: slangley at google dot com
Reported by: slangley at google dot com
Summary: Passing > 1 additional module to php_module_startup
causes a segfault
Status: Feedback
Type: Bug
Package: Reproducible crash
Operating System: N/A
PHP Version: 5.4.7
Block user comment: N
Private report: N
New Comment:
Suggest you look again - it is not possible to get past this line without
segfaulting if there is more than one extension.
https://github.com/php/php-src/blob/master/main/main.c#L2191
Previous Comments:
------------------------------------------------------------------------
[2013-01-28 09:41:28] [email protected]
Works just fine here.
Which architecture do you use? Maybe some rounding issues?
However I would suggest you to manually set the modules count to the right
numbers
instead, to avoid any kind of architecture specific troubles.
------------------------------------------------------------------------
[2013-01-28 09:21:38] slangley at google dot com
Sure - this will result in a segfault.
int register_moules(sapi_module_struct* sapi_module) {
zend_module_entry modules[] = {
my_module_1_entry,
my_module_2_entry,
};
int module_count = sizeof(modules) / sizeof(zend_module_entry);
return php_module_startup(sapi_module, modules, module_count);
}
------------------------------------------------------------------------
[2013-01-28 00:56:16] [email protected]
Could you add some (short) code that would help to reproduce the problem?
------------------------------------------------------------------------
[2012-09-25 10:34:03] slangley at google dot com
Not sure how that might be ...
Here's the signature for php_module_startup
int php_module_startup(sapi_module_struct *sf, zend_module_entry
*additional_modules, uint num_additional_modules);
The parameter num_additional_modules suggests that more then one module_entry
can be passed in the additional_modules argument, but it's only a pointer not
an
array of pointers.
Here's the signature for php_register_extensions, which is called from
php_module_startup.
int php_register_extensions(zend_module_entry **ptr, int count TSRMLS_DC);
This more correctly takes an array of pointers.
I don't know why this wasn't fixed when first reported in 2004 - I couldn't
find
any more information than the thread I linked.
------------------------------------------------------------------------
[2012-09-25 09:42:22] [email protected]
I am not sure, is this just a wrong usage of ZEND API?
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=63159
--
Edit this bug report at https://bugs.php.net/bug.php?id=63159&edit=1
