Bug 59617 - Crash in isReachableFromOpaqueRoots when loading versiontracker.com
Summary: Crash in isReachableFromOpaqueRoots when loading versiontracker.com
Status: RESOLVED WORKSFORME
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore JavaScript (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac (Intel) OS X 10.6
: P1 Critical
Assignee: Nobody
URL: http://versiontracker.com/macosx/
Keywords: Regression
Depends on:
Blocks:
 
Reported: 2011-04-27 10:00 PDT by email.webkit
Modified: 2011-05-03 13:38 PDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description email.webkit 2011-04-27 10:00:54 PDT
Any visit to http://versiontracker.com/macosx/ (which redirects to a download page on CNet) causes latest Webkit to crash. In Safari, it functions fine.  Sometimes the first page comes up with the popup that reads "Versiontracker.com is now part of CNet" and then it crashes when you click to dismiss it, other times it crashes instantly by entering that url.
Comment 1 email.webkit 2011-04-27 10:04:14 PDT
The URL it redirects to is:
http://www.versiontracker.com/macosx/ First
then to
http://download.cnet.com/mac/3151-20_4-0.html?tag=vtredir
Navigating to that URL causes the same crash to occur with either instant crash or when clicking to dismiss the overlaid box.
Comment 2 Alexey Proskuryakov 2011-04-27 14:15:23 PDT
I get the crash with r84622 nightly, bit not with a local debug build of r85060. Geoff, was this fixed already?

Thread 0 Crashed:  Dispatch queue: com.apple.main-thread
0   com.apple.WebCore             	0x00000001011ec4b3 WebCore::JSCSSStyleDeclarationOwner::isReachableFromOpaqueRoots(JSC::Handle<JSC::Unknown>, void*, JSC::MarkStack&) + 35
1   com.apple.JavaScriptCore      	0x00000001007fbdb8 JSC::HandleHeap::markWeakHandles(JSC::HeapRootVisitor&) + 120
Comment 3 Geoffrey Garen 2011-04-28 11:51:09 PDT
(In reply to comment #2)
> I get the crash with r84622 nightly, bit not with a local debug build of r85060. Geoff, was this fixed already?

Hmmm... I don't recall a patch specifically targeted at JSCSSStyleDeclarationOwner::isReachableFromOpaqueRoots.
Comment 4 Alexey Proskuryakov 2011-04-28 14:32:27 PDT
I cannot reproduce with a release build of r85222 either.

There was a number of generic GC fixes too, so marking as WORKSFORME. Could you please try to reproduce when a new nightly is posted?
Comment 5 Geoffrey Garen 2011-05-03 13:37:05 PDT
(FYI, this particular website was fixed by http://trac.webkit.org/changeset/84764.)
Comment 6 Geoffrey Garen 2011-05-03 13:38:00 PDT
<rdar://problem/9370087>