[webkit-reviews] review granted: [Bug 125110] Possible crash in void ProgressTracker::progressHeartbeatTimerFired(Timer<ProgressTracker>*) : [Attachment 218223] Patch v1

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Dec 2 15:24:20 PST 2013


Darin Adler <darin at apple.com> has granted Brady Eidson <beidson at apple.com>'s
request for review:
Bug 125110: Possible crash in void
ProgressTracker::progressHeartbeatTimerFired(Timer<ProgressTracker>*)
https://bugs.webkit.org/show_bug.cgi?id=125110

Attachment 218223: Patch v1
https://bugs.webkit.org/attachment.cgi?id=218223&action=review

------- Additional Comments from Darin Adler <darin at apple.com>
While there is no harm in adding a null check, if this is actually a race
condition, then a null check is likely not sufficient to fix the problem. It
might, however, reduce the frequency of a crash.


More information about the webkit-reviews mailing list