[Webkit-unassigned] [Bug 19588] CRASH doing open() on destroyed window

Tue Jul 22 14:21:06 PDT 2008

https://bugs.webkit.org/show_bug.cgi?id=19588

jchaffraix at webkit.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #22432|                            |review?
               Flag|                            |

------- Comment #8 from jchaffraix at webkit.org  2008-07-22 14:21 PDT -------
Created an attachment (id=22432)
 --> (https://bugs.webkit.org/attachment.cgi?id=22432&action=view)
Updated version: use top() and cover the other crash pointed by Eric

> Why shouldn't you just use "top()" here?  It seems "top()" wouldn't crash.

top() is indeed fine.

> Seems like the end of the function doesn't check against null pages either...
> so that will need to be fixed and a test case added too.

True, I have fixed that too including a layout test.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.