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

 ID:                 62852
 Patch added by:     re...@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] larue...@php.net

closed by commit email, reopen

------------------------------------------------------------------------
[2012-09-15 04:26:07] re...@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] larue...@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] larue...@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] larue...@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

Reply via email to