Bug #72209 [NEW]: ReflectionProperty#getValue() doesn't fail if object doesn't match type
| From: | ocramius at gmail dot com | Date: | Thu, 12 May 2016 15:37:51 +0000 |
| Subject: | Bug #72209 [NEW]: ReflectionProperty#getValue() doesn't fail if object doesn't match type | ||
| Groups: | php.bugs | ||
| Request: | Send a blank email to php-bugs+get-201043@lists.php.net to get a copy of this message | ||
From: ocramius at gmail dot com
Operating system:
PHP version: 7.0.7RC1
Package: Reflection related
Bug Type: Bug
Bug description:ReflectionProperty#getValue() doesn't fail if object doesn't match type
Description:
------------
ReflectionProperty#getValue($object) does not check whether the given
$object matches the expected type.
If $r = ReflectionProperty(Foo::class, 'bar'), then only instances of
Foo should be allowed by ReflectionProperty#getValue() and
ReflectionProperty#setValue(). All other values should cause an
exception to be raised.
As a reference, ReflectionMethod behaves correctly (
https://3v4l.org/U2u0h ), since
ReflectionMethod#invoke($object) and
ReflectionMethod#invokeArgs($object) reject incompatible $object
instances.
Test script:
---------------
<?php
class Foo
{
public $bar = __LINE__;
}
class Baz
{
public $bar = __LINE__;
}
var_dump((new ReflectionProperty(Foo::class, 'bar'))->getValue(new
Baz()));
Expected result:
----------------
Fatal error: Uncaught ReflectionException: Expected an instance of Foo,
got 'Baz' instead does not exist in /.../test-script.php
Actual result:
--------------
(int) 10
--
Edit bug report at https://bugs.php.net/bug.php?id=72209&edit=1
--
Try a snapshot (PHP 5.4): https://bugs.php.net/fix.php?id=72209&r=trysnapshot54
Try a snapshot (PHP 5.5): https://bugs.php.net/fix.php?id=72209&r=trysnapshot55
Try a snapshot (trunk): https://bugs.php.net/fix.php?id=72209&r=trysnapshottrunk
Fixed in SVN: https://bugs.php.net/fix.php?id=72209&r=fixed
Fixed in release: https://bugs.php.net/fix.php?id=72209&r=alreadyfixed
Need backtrace: https://bugs.php.net/fix.php?id=72209&r=needtrace
Need Reproduce Script: https://bugs.php.net/fix.php?id=72209&r=needscript
Try newer version: https://bugs.php.net/fix.php?id=72209&r=oldversion
Not developer issue: https://bugs.php.net/fix.php?id=72209&r=support
Expected behavior: https://bugs.php.net/fix.php?id=72209&r=notwrong
Not enough info: https://bugs.php.net/fix.php?id=72209&r=notenoughinfo
Submitted twice: https://bugs.php.net/fix.php?id=72209&r=submittedtwice
register_globals: https://bugs.php.net/fix.php?id=72209&r=globals
PHP 4 support discontinued: https://bugs.php.net/fix.php?id=72209&r=php4
Daylight Savings: https://bugs.php.net/fix.php?id=72209&r=dst
IIS Stability: https://bugs.php.net/fix.php?id=72209&r=isapi
Install GNU Sed: https://bugs.php.net/fix.php?id=72209&r=gnused
Floating point limitations: https://bugs.php.net/fix.php?id=72209&r=float
No Zend Extensions: https://bugs.php.net/fix.php?id=72209&r=nozend
MySQL Configuration Error: https://bugs.php.net/fix.php?id=72209&r=mysqlcfg