Bug #63159 [Com]: Passing > 1 additional module to php_module_startup causes a segfault

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] pajoye@php.net 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] stas@php.net 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] laruence@php.net 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