|  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #60919 __wakeup is not called
Submitted: 2012-01-28 20:58 UTC Modified: 2012-02-23 18:34 UTC
Avg. Score:5.0 ± 0.0
Reproduced:2 of 2 (100.0%)
Same Version:1 (50.0%)
Same OS:2 (100.0%)
From: Assigned: stas (profile)
Status: Duplicate Package: Scripting Engine problem
PHP Version: 5.4.0RC6+ OS: (Any)
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome! If you don't have a Git 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.
Block user comment
Status: Assign to:
Bug Type:
New email:
PHP Version: OS:


 [2012-01-28 20:58 UTC]
In 5.4.0RC6, a __wakeup() method is no longer called during unserialization.

Test script:
$ php -r 'class A { function __sleep() { echo "SLEEP\n"; return array(); } function __wakeup() { echo "WAKEUP"; }} $a= unserialize(serialize(new A()));'

Expected result:
object(A)#1 (0) {

Actual result:
object(A)#1 (0) {


init-serialize-levels.diff (last revision 2012-02-05 14:03 UTC by

Add a Patch

Pull Requests

Add a Pull Request


AllCommentsChangesGit/SVN commitsRelated reports
 [2012-01-28 21:15 UTC]
-Status: Open +Status: Critical
 [2012-01-28 21:23 UTC]
-Status: Critical +Status: Feedback
 [2012-01-28 21:23 UTC]
I can't actually reproduce this in the current 5.4 code:

% php --version
PHP 5.4.0RC7-dev (cli) (built: Jan 28 2012 13:21:30) 
Copyright (c) 1997-2012 The PHP Group
Zend Engine v2.4.0, Copyright (c) 1998-2012 Zend Technologies

% php -r 'class A { function __sleep() { echo "SLEEP\n"; return array(); } 
function __wakeup() { echo "WAKEUP"; }} $a= unserialize(serialize(new A()));'
 [2012-02-04 17:47 UTC]
-PHP Version: 5.4.0RC6 +PHP Version: 5.4.0RC6+
 [2012-02-04 17:47 UTC]
But still occurs in php-5.4.0RC7-nts-Win32-VC9-x86 (2012-Feb-02 23:15:00)
 [2012-02-05 14:00 UTC]
I can also not reproduce the issue if I compile PHP myself, neither on Linux nor on Windows.

From looking at the diffs between 5.4.0RC5 and current SVN, BG(serialize_lock) in conjunction with BG(unserialize).level and BG(serialize).level) sticks out. The thing I think is suspicous here is that BG(unserialize).level gets assigned to 1 inside a block guarded by if (... || !BG(unserialize).level) {, but is never assigned to 0. 

Are we running into this problem because BG(unserialize).level doesn't properly get initialized? I'm not an expert here but is it safe to rely on the compiler doing thos?

Same goes for BG(serialize).level, by the way.
 [2012-02-05 14:03 UTC]
The following patch has been added/updated:

Patch Name: init-serialize-levels.diff
Revision:   1328450632
 [2012-02-05 16:28 UTC]
You should ask Stas for permission to commit this patch to 5.4 before the 
release. Initializing uninitialized variables tends to be a good idea.
 [2012-02-05 18:02 UTC]
-Assigned To: +Assigned To: stas
 [2012-02-05 18:02 UTC]
Stas, can you have a look at the patch? Thanks!
 [2012-02-20 10:48 UTC]
Duplicate of #60879
 [2012-02-20 10:48 UTC]
-Status: Feedback +Status: Duplicate -Assigned To: stas +Assigned To: pajoye
 [2012-02-20 10:52 UTC]
-Status: Duplicate +Status: Feedback -Assigned To: pajoye +Assigned To: stas
 [2012-02-20 10:52 UTC]
Actually maybe still valid on other platform than windows, but so far our tests on 
linux (x86&x64) did not show any issue with this issue.

Timm, which platforms exactly do you use to test? The other bug I referred to 
(#60879) is a windows only bug and only due to the release build process, not to a 
bug in our code base.
 [2012-02-23 18:34 UTC]
-Status: Feedback +Status: Duplicate
 [2012-02-23 18:34 UTC]
Yes, also looks like a dup of #60879 to me; and the patched RC8 there works for me. I'll close this.
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Thu Feb 22 12:01:29 2024 UTC