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

 ID:                 60919
 Updated by:         [email protected]
 Reported by:        [email protected]
 Summary:            __wakeup is not called
-Status:             Feedback
+Status:             Duplicate
 Type:               Bug
 Package:            Scripting Engine problem
 Operating System:   (Any)
 PHP Version:        5.4.0RC6+
-Assigned To:        stas
+Assigned To:        pajoye
 Block user comment: N
 Private report:     N

 New Comment:

Duplicate of #60879


Previous Comments:
------------------------------------------------------------------------
[2012-02-05 18:02:48] [email protected]

Stas, can you have a look at the patch? Thanks!

------------------------------------------------------------------------
[2012-02-05 16:28:20] [email protected]

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 14:03:53] [email protected]

The following patch has been added/updated:

Patch Name: init-serialize-levels.diff
Revision:   1328450632
URL:        
https://bugs.php.net/patch-display.php?bug=60919&patch=init-serialize-levels.diff&revision=1328450632

------------------------------------------------------------------------
[2012-02-05 14:00:32] [email protected]

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-04 17:47:12] [email protected]

But still occurs in php-5.4.0RC7-nts-Win32-VC9-x86 (2012-Feb-02 23:15:00)

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


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


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

Reply via email to