Unserialize в php 5.4.29/5.5.13 работает не так, как прежде

[hell0w0rd]

В общем такой код выдает теперь warning. Это код из доктрины, не знаю, где еще подобное используется.

unserialize(sprintf('O:%d:"%s":0:{}', strlen($this->name), $this->name));

Вот предложенный фикс:

if (PHP_VERSION_ID === 50429 || PHP_VERSION_ID === 50513) {
    $this->_prototype = $this->reflClass->newInstanceWithoutConstructor();
} else {
    $this->_prototype = unserialize(sprintf('O:%d:"%s":0:{}', strlen($this->name), $this->name));
}

Так что если у вас в проекте такое есть - исправьте, а кто юзает доктрину - не советую php обновлять

 

[Вурдалак]

Так может быть там стоит поставить PHP_VERSION_ID >=? А что за warning?

 

[флоппик]

А оно работало ли вообще нормально?

ubuntu@server:~$ php -v
PHP 5.3.10-1ubuntu3.11 with Suhosin-Patch (cli) (built: Apr  4 2014 01:30:04)
Copyright (c) 1997-2012 The PHP Group
Zend Engine v2.3.0, Copyright (c) 1998-2012 Zend Technologies
    with XCache v1.3.2, Copyright (c) 2005-2011, by mOo

ubuntu@server:~$ php -r "echo unserialize(sprintf('O:%d:"%s":0:{}', strlen('test'), 'test'));"

PHP Notice:  unserialize(): Error at offset 0 of 13 bytes in Command line run code on line 1

 

[tony2001]

в скрипт положи, в консоли кавычки всё время путаются.

 

[флоппик]

а, точно, сорри.

 

[keltanas]

hell0w0rd, а где это в доктрине используется и для чего?

 

[hell0w0rd]

keltanas, когда она грузит данные из базы и рассовывает по объектам. Она это делает в обход коснтруктора, а такие выкрутасы доступны с 5.4 версии.
Возможно такое решение существует из-за скорости, но мне кажется просто участок кода "работает - не трогай".
Вурдалак, это фикс с stackoverflow. На мой взгляд он должен выглядеть в сравнении с 50400, ибо там уже доступен метод для рефлексии.
Варнинг:

Warning: Erroneous data format for unserializing 'App\UserBundle\Entity\User' in project/vendor/doctrine/orm/lib/Doctrine/ORM/Mapping/ClassMetadataInfo.php line 869

 

[hell0w0rd]

флоппик, вместо php -r для тестов использую php -a, если влом файл создавать http://www.php.net/manual/ru/features.commandline.interactive.php

 

[hell0w0rd]

Кому все еще интересно - в доктрину весит 3-4 PR (один из которых мой), исправляющих баг, но ни один не хотят принимать.
А тот, где сравнение версий вхардкодено, так и не выпустили релиз)) Так что не используйте доктрину

 

[keltanas]

Так что не используйте доктрину

Ага, давайте теперь на голом PDO работать )) Или вообще какое-нибудь фуйло юзать )))

 

[hell0w0rd]

keltanas, версию они таки выпустили и пример мне показали, где не работает рефлексия. Так что ты прав.

 

[hell0w0rd]

В общем ребята из доктрины приняли предложенный фикс, который конечно работает на определенных версиях и фейлится в 5.6.0, ждем 2.5.0, там всю эту байду в отдельный класс выделили

 

[fixxxer]

Чета какой-то булшит, а не фикс.

Надо проверять на наличие newInstanceWithoutConstructor, а если его нет, использовать говнохак с unserialize, все будет ок

 

[hell0w0rd]

fixxxer, да они там вообще зафигачили отдельный компонент, с кешем инстансов и поэтессами: https://github.com/doctrine/instantiator/blob/master/src/Doctrine/Instantiator/Instantiator.php

 

[Redjik]

что такое поэтесса?

 

[hell0w0rd]

Redjik, http://lurkmore.to/_/7430#mws_FV0JxKd