php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Doc Bug #49826 Code example known to be exploitable.
Submitted: 2009-10-09 21:18 UTC Modified: 2009-11-13 21:12 UTC
From: kore@php.net Assigned: ilia (profile)
Status: Closed Package: Documentation problem
PHP Version: Irrelevant OS: Irrelevant
Private report: No CVE-ID: None
 [2009-10-09 21:18 UTC] kore@php.net
Description:
------------
Example #2 on http://docs.php.net/manual/en/security.magicquotes.disabling.php uses source, which is known to be exploitable, like shown here: http://ilia.ws/files/phpworks_security.pdf (Slides 18 to 20)


Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2009-10-09 23:09 UTC] bjori@php.net
I think the argument about crashing php because it used recursive function is a total crap.
There was a 'security fix' recently (the max_input_nesting_level INI setting) to fix exactly that, and no Ilia was not the one who reported it nor fixed it:
 Added "max_input_nesting_level" php.ini option to limit
 nesting level of input variables. Fix for MOPB-03-2007. (Stas).

And the code on the slide is worse AFAICT, it doesn't deal with arrays nor $_REQUEST at all, so AFAICT it introduces security issues, not fixes them.


Ilia: please have a look at the example and clarify if needed..
 [2009-11-13 21:12 UTC] vrana@php.net
This bug has been fixed in the documentation's XML sources. Since the
online and downloadable versions of the documentation need some time
to get updated, we would like to ask you to be a bit patient.

Thank you for the report, and for helping us make our documentation better.


 [2009-11-13 21:12 UTC] svn@php.net
Automatic comment from SVN on behalf of vrana
Revision: http://svn.php.net/viewvc/?view=revision&revision=290699
Log: Improve run-time disabling of magic quotes (bug #49826)
 
PHP Copyright © 2001-2026 The PHP Group
All rights reserved.
Last updated: Wed Jun 17 16:00:02 2026 UTC