Closed Bug 309702 Opened 19 years ago Closed 19 years ago

Crashes when adblocking is enabled [@ nsPluginStreamListener::OnStartRequest]

Categories

(Core :: DOM: Core & HTML, defect)

PowerPC
macOS
defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla1.9alpha1

People

(Reporter: sugar.waffle, Assigned: Biesinger)

References

()

Details

(Keywords: crash, regression)

Crash Data

Attachments

(3 files)

Incident ID: TB9634495X, TB9634658E, TB9634714G It crashes by some pages when adblock is enable. Reproducible: Always Steps to Reproduce: 1. Adblock is turned on with Preferences. 2. open URL 3. The following links are clicked with a left frame. http://live16.2ch.net/liventv/ Mac OS X 10.3.9 trunk build: 2005092208 (v1.0+)
Attached file crash log
Keywords: crash
Summary: It crashes when adblock is enable. → Crashes when adblocking is enabled [@ nsPluginStreamListener::OnStartRequest]
the crash is in the plugin code. why would ad blocking affect that?
(In reply to comment #2) > the crash is in the plugin code. why would ad blocking affect that? Maybe we block a Flash plugin, and an iframe ends up with no content or something. I wasn't able to find the liventv link in the left frame (too many links!).
Attached patch possible patchSplinter Review
this should fix it, I believe...
Assignee: mikepinkerton → cbiesinger
Status: UNCONFIRMED → ASSIGNED
Attachment #197188 - Flags: superreview?(bzbarsky)
Attachment #197188 - Flags: review?(bzbarsky)
Can we at least get a testcase?
Attachment #197188 - Flags: superreview?(bzbarsky)
Attachment #197188 - Flags: superreview+
Attachment #197188 - Flags: review?(bzbarsky)
Attachment #197188 - Flags: review+
Testcase. This crashes in nsPluginStreamListener::OnStartRequest so I believe it's this bug. (any my patch does fix it)
Component: General → DOM
Flags: review+
Product: Camino → Core
Checking in content/html/document/src/nsPluginDocument.cpp; /cvsroot/mozilla/content/html/document/src/nsPluginDocument.cpp,v <-- nsPluginDocument.cpp new revision: 1.23; previous revision: 1.22 done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9alpha
(what I fixed was caused by bug 1156)
Blocks: 1156
Do we need this on the branch?
no, bug 1156 was trunk only.
Crash Signature: [@ nsPluginStreamListener::OnStartRequest]
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: