#38018 [NEW]: __destruct called if __wakeup threw an exception

From: php dot bugs at login dot hedim dot ch Operating system: FreeBSD PHP version: 5.1.4 PHP Bug Type: Class/Object related Bug description: __destruct called if __wakeup threw an exception Description: ------------ If __wakeup threw an exception when unserializing an object, __destruct is called on that object. For the same reasons as __destruct is not called if __construct threw an exception (see bug #29368), it should not be called here either. Reproduce code: --------------- class Foo { function __wakeup() { throw new Exception; } function __destruct() { echo "hello from destructor\n"; } } unserialize(serialize(new Foo)); Expected result: ---------------- hello from destructor (from the destructor of the object to be serialized) Actual result: -------------- hello from destructor (from the destructor of the object to be serialized) hello from destructor (from the destructor of the object that fails to unserialize) -- Edit bug report at http://bugs.php.net/?id=38018&edit=1 -- Try a CVS snapshot (PHP 4.4): http://bugs.php.net/fix.php?id=38018&r=trysnapshot44 Try a CVS snapshot (PHP 5.2): http://bugs.php.net/fix.php?id=38018&r=trysnapshot52 Try a CVS snapshot (PHP 6.0): http://bugs.php.net/fix.php?id=38018&r=trysnapshot60 Fixed in CVS: http://bugs.php.net/fix.php?id=38018&r=fixedcvs Fixed in release: http://bugs.php.net/fix.php?id=38018&r=alreadyfixed Need backtrace: http://bugs.php.net/fix.php?id=38018&r=needtrace Need Reproduce Script: http://bugs.php.net/fix.php?id=38018&r=needscript Try newer version: http://bugs.php.net/fix.php?id=38018&r=oldversion Not developer issue: http://bugs.php.net/fix.php?id=38018&r=support Expected behavior: http://bugs.php.net/fix.php?id=38018&r=notwrong Not enough info: http://bugs.php.net/fix.php?id=38018&r=notenoughinfo Submitted twice: http://bugs.php.net/fix.php?id=38018&r=submittedtwice register_globals: http://bugs.php.net/fix.php?id=38018&r=globals PHP 3 support discontinued: http://bugs.php.net/fix.php?id=38018&r=php3 Daylight Savings: http://bugs.php.net/fix.php?id=38018&r=dst IIS Stability: http://bugs.php.net/fix.php?id=38018&r=isapi Install GNU Sed: http://bugs.php.net/fix.php?id=38018&r=gnused Floating point limitations: http://bugs.php.net/fix.php?id=38018&r=float No Zend Extensions: http://bugs.php.net/fix.php?id=38018&r=nozend MySQL Configuration Error: http://bugs.php.net/fix.php?id=38018&r=mysqlcfg