Edit report at https://bugs.php.net/bug.php?id=62852&edit=1
ID: 62852
Patch added by: reeze@php.net
Reported by: kasper at webmasteren dot eu
Summary: Unserialize Invalid Date causes crash
Status: Re-Opened
Type: Bug
Package: Reproducible crash
Operating System: windows, linux
PHP Version: Irrelevant
Assigned To: laruence
Block user comment: N
Private report: N
New Comment:
The following patch has been added/updated:
Patch Name: Fix-add-exception-checking
Revision: 1347761929
URL: https://bugs.php.net/patch-display.php?bug=62852&patch=Fix-add-exception-checking&revision=1347761929
Previous Comments:
------------------------------------------------------------------------
[2012-09-15 06:57:20] laruence@php.net
closed by commit email, reopen
------------------------------------------------------------------------
[2012-09-15 04:26:07] reeze@php.net
Yeah, the segfault is bad.
but the test script is wired, why do you want to refer to it before wakeup?
When construct the DateTime if invalid date it throw exception, so
when unserialize from an invalid date throw exception is reasonable.
------------------------------------------------------------------------
[2012-09-15 03:33:26] laruence@php.net
Automatic comment on behalf of laruence
Revision: http://git.php.net/?p=php-src.git;a=commit;h=e766f85405cd936a07a30a045f419199b6c02ed7
Log: Revert "Fixed bug #62852 (Unserialize invalid DateTime causes crash)"
------------------------------------------------------------------------
[2012-09-15 03:32:53] laruence@php.net
Automatic comment on behalf of laruence
Revision: http://git.php.net/?p=php-src.git;a=commit;h=e766f85405cd936a07a30a045f419199b6c02ed7
Log: Revert "Fixed bug #62852 (Unserialize invalid DateTime causes crash)"
------------------------------------------------------------------------
[2012-09-15 03:30:56] laruence@php.net
@tstarling okey, I reverted. and make the test XFAIL for now, we should fix this
in another way.
------------------------------------------------------------------------
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=62852
--
Edit this bug report at https://bugs.php.net/bug.php?id=62852&edit=1