PHP :: Bug #68260 :: SQLite3Result::fetchArray declares wrong required_num_args
| Bug #68260 | SQLite3Result::fetchArray declares wrong required_num_args | ||||
|---|---|---|---|---|---|
| Submitted: | 2014-10-18 21:32 UTC | Modified: | 2015-01-09 15:28 UTC | ||
| From: | stesie at brokenpipe dot de | Assigned: | jpauli (profile) | ||
| Status: | Closed | Package: | SQLite related | ||
| PHP Version: | master-Git-2014-10-18 (Git) | OS: | unrelated | ||
| Private report: | No | CVE-ID: | None | ||
[2014-10-18 21:32 UTC] stesie at brokenpipe dot de
Description: ------------ The SQLite3Result::fetchArray has on *optional* argument, the implementation also handles it as an optional argument. However the corresponding ARG_INFO_EX block declares a mandatory argument (4th macro parameter): --- BEGIN --- ZEND_BEGIN_ARG_INFO_EX(arginfo_sqlite3result_fetcharray, 0, 0, 1) ZEND_ARG_INFO(0, mode) ZEND_END_ARG_INFO() --- END --- This does *not* cause problems with the PHP interpreter itself, since it allows calling a function with to few arguments ... after all it's up to the called function to handle that situation well. However there are extensions (namely "V8Js"), that aim at sandboxing user code and hence apply stricter rules. Namely V8Js throws an exception in case too few arguments are provided. Therefore I ask you to please fix the arg info. Link to bug report against V8Js: https://github.com/preillyme/v8js/issues/26 cheers ~stesie
Patches
fix-arginfo.diff (last revision 2014-10-18 21:32 UTC by stesie at brokenpipe dot de)Pull Requests
History
AllCommentsChangesGit/SVN commits
[2015-01-09 15:27 UTC] jpauli@php.net
-Status: Closed +Status: Feedback
[2015-01-09 15:28 UTC] jpauli@php.net
-Status: Feedback +Status: Closed -Assigned To: +Assigned To: jpauli