Bug #55475 [Asn]: is_a() triggers autoloader

From: Date: Tue, 20 Sep 2011 21:30:06 +0000
Subject: Bug #55475 [Asn]: is_a() triggers autoloader
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-163728@lists.php.net to get a copy of this message 
Edit report at https://bugs.php.net/bug.php?id=55475&edit=1

 ID:                 55475
 Updated by:         alan_k@php.net
 Reported by:        mads at gartneriet dot dk
 Summary:            is_a() triggers autoloader
 Status:             Assigned
 Type:               Bug
 Package:            Scripting Engine problem
 PHP Version:        5.3.7
 Assigned To:        dmitry
 Block user comment: N
 Private report:     N

 New Comment:

Attached now is a patch that fixes this by adding

is_class_of

Which behaves the same as is_subclass_of, (autoload/ accepts strings)

It also fixes the documentation on is_subclass_of and reverts the behaviour of 
is_a

Note: the is_a change is now a security bug as sending url's to is_a may trigger 
remote code execution now.

Note: I'm not sure you can classify a 2-3 developers comments as the "common 
decision" there where objections to the original change from core developers, 
this patch gives everybody what they want....


Previous Comments:
------------------------------------------------------------------------
[2011-09-20 21:25:58] alan_k@php.net

The following patch has been added/updated:

Patch Name: is_class_of.txt
Revision:   1316553958
URL:        https://bugs.php.net/patch-display.php?bug=55475&patch=is_class_of.txt&revision=1316553958

------------------------------------------------------------------------
[2011-09-15 11:00:29] dmitry@php.net

Reverted before the common decision.

------------------------------------------------------------------------
[2011-09-15 10:59:23] dmitry@php.net

Automatic comment from SVN on behalf of dmitry
Revision: http://svn.php.net/viewvc/?view=revision&revision=316811
Log: Reverted the fix for #55475 (is_a() triggers autoloader) before the common decision

------------------------------------------------------------------------
[2011-09-15 10:00:16] dmitry@php.net

I've committed the revert.is_a.behaviour.to.ignoring.strings.diff by alan at akbkhome dot com
into 5.3.

5.4 is going to support string argument.

------------------------------------------------------------------------
[2011-09-15 09:58:17] dmitry@php.net

Automatic comment from SVN on behalf of dmitry
Revision: http://svn.php.net/viewvc/?view=revision&revision=316810
Log: Fixed bug #55475 (is_a() triggers autoloader). (alan at akbkhome dot com)

------------------------------------------------------------------------


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=55475


-- 
Edit this bug report at https://bugs.php.net/bug.php?id=55475&edit=1

Thread (51 messages)

« previous php.bugs (#163728) next »