Bug 7808 - Assertion failure in -[WebBaseNetscapePluginStream dealloc] when requesting an invalid URL
Summary: Assertion failure in -[WebBaseNetscapePluginStream dealloc] when requesting a...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Plug-ins (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P1 Normal
Assignee: Nobody
URL: http://www.wdsu.com/video/index.html
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2006-03-16 13:32 PST by Gibbons Burke
Modified: 2006-07-11 11:52 PDT (History)
3 users (show)

See Also:


Attachments
fix assertion failure (2.48 KB, patch)
2006-07-11 10:26 PDT, Alexey Proskuryakov
timo: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gibbons Burke 2006-03-16 13:32:44 PST
The web page listed in the URL field freezes Safari and the Nightly Build Webkit app every time I try and bring it up. I get the spinning beachball before the page finishes loading and the following messages appear in my Console log: 

(event handler):Undefined value
(event handler):Undefined value
[1694] http://www.wdsu.com/video/index.html line 40: ReferenceError: Can't find variable: videoplug
[1694] http://www.wdsu.com/video/index.html line 44: TypeError: Undefined value
[1694] http://www.wdsu.com/video/index.html line 50: ReferenceError: Can't find variable: NewState
Comment 1 Richard Das 2006-03-24 07:16:56 PST
Same here. Looks like same behavior as when rapidly clicking several links in an rss feed to open in new tabs. Pages start to load, then you get the beach ball.
Comment 2 Alexey Proskuryakov 2006-03-24 13:10:49 PST
=================
ASSERTION FAILED: !instance (/Users/ap/WebKit/WebKit/Plugins/WebBaseNetscapePluginStream.m:107 -[WebBaseNetscapePluginStream dealloc])
=================

0   com.apple.WebKit               	0x0031b640 -[WebBaseNetscapePluginStream dealloc] + 96 (WebBaseNetscapePluginStream.m:107)
1   com.apple.WebKit               	0x0032cd34 -[WebNetscapePluginStream dealloc] + 160 (WebNetscapePluginStream.m:99)
2   com.apple.WebKit               	0x0032cb3c -[WebNetscapePluginStream initWithRequest:pluginPointer:notifyData:sendNotification:] + 472 (WebNetscapePluginStream.m:78)
3   com.apple.WebKit               	0x00324ca0 -[WebBaseNetscapePluginView(WebNPPCallbacks) loadRequest:inTarget:withNotifyData:sendNotification:] + 872 (WebBaseNetscapePluginView.m:1603)
4   com.apple.WebKit               	0x00324ec8 -[WebBaseNetscapePluginView(WebNPPCallbacks) getURL:target:] + 176 (WebBaseNetscapePluginView.m:1628)
5   com.apple.WebKit               	0x003314d4 NPN_GetURL + 88 (npapi.m:88)
6   Windows Media Plugin           	0x13217968 0x13205000 + 76136
7   Windows Media Plugin           	0x13218c70 0x13205000 + 81008
8   Windows Media Plugin           	0x13217ab0 0x13205000 + 76464
9   com.apple.WebKit               	0x00321944 -[WebBaseNetscapePluginView start] + 676 (WebBaseNetscapePluginView.m:1000)
10  com.apple.WebKit               	0x00323278 -[WebBaseNetscapePluginView viewDidMoveToWindow] + 132 (WebBaseNetscapePluginView.m:1306)
Comment 3 Alexey Proskuryakov 2006-03-24 13:15:55 PST
Forgot to mention that I cannot reproduce the freeze with stock 2.0.3/10.4.5, but the assertion failure looks scary enough. See also: bug 7762.
Comment 4 Eric Seidel (no email) 2006-03-26 16:02:00 PST
The subframe scrollbar on the right hand side (list of videos) doesn't seem to work on 2.0.3  However I don't see the page hang in 2.0.3.
Comment 5 Darin Adler 2006-04-27 19:56:29 PDT
Looks like the assertion is related to GC changes to plug-in classes.
Comment 6 Alice Liu 2006-05-16 09:42:22 PDT
<rdar://problem/4549734>
Comment 7 Darin Adler 2006-06-04 11:37:02 PDT
After doing some quick analysis, this looks like the event listener problem that's also causing a hang on the digg.com site, bug 7657.
Comment 8 Darin Adler 2006-06-04 11:38:58 PDT
Or maybe just a runaway for loop.
Comment 9 Alexey Proskuryakov 2006-07-11 10:26:30 PDT
Created attachment 9370 [details]
fix assertion failure

The plugin was requesting an invalid (empty) URL, and this fix just removes an early return in this case. Not sure why it was there, NSURLConnection seems to handle this nicely. BTW, looks scarily similar to the completely unrelated bug 9802 :)

As noted in comment 4, scrollbars in subframes don't work, but this is certainly a separate issue.
Comment 10 Tim Omernick 2006-07-11 10:55:11 PDT
Comment on attachment 9370 [details]
fix assertion failure

Looks good!  r=me
Comment 11 Alexey Proskuryakov 2006-07-11 11:23:32 PDT
Committed revision 15337. Filed the scroll bars issue as bug 9855. Please file additional bugs if there are more problems remaining on this site.
Comment 12 Tim Omernick 2006-07-11 11:52:08 PDT
(In reply to comment #6)
> <rdar://problem/4549734>
> 


This does *not* appear to be the same as Radar 4549734.  It looks much more like Radar 4594198.