PHP Bugs  
php.net | support | documentation | report a bug | advanced search | search howto | statistics | login

go to bug id or search bugs for  

Bug #49826 Code example known to be exploitable.
Submitted:9 Oct 9:18pm UTC Modified: 13 Nov 9:12pm UTC
From:kore@php.net Assigned to:ilia
Status:Closed Category:Documentation problem
Version:Irrelevant OS:Irrelevant
View/Vote Developer Edit Submission

Welcome! If you don't have a SVN account, you can't do anything here. You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
SVN Username: SVN Password:
Quick Fix:
Status: Assign to:
Category:
Summary:
From: kore@php.net
New email:
Version: OS:
New/Additional Comment:

[9 Oct 9:18pm 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)

[9 Oct 11:09pm 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..
[13 Nov 9:12pm 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.

[13 Nov 9:12pm 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)

RSS feed | show source 

PHP Copyright © 2001-2009 The PHP Group
All rights reserved.
Last updated: Sat Nov 21 10:30:49 2009 UTC