Closed Bug 441559 Opened 17 years ago Closed 16 years ago

DOM modification using JavaScript causes Access Violation [@ nsFileControlFrame::CreateAnonymousContent] (appears to be NULL pointer)

Categories

(Core :: Layout, defect)

1.8 Branch
x86
Windows Vista
defect
Not set
critical

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: u315569, Unassigned)

References

()

Details

(Keywords: crash, testcase, Whiteboard: [sg:dos])

Crash Data

Attachments

(2 files)

User-Agent: Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 6.0; WOW64; SLCC1; .NET CLR 2.0.50727; .NET CLR 3.0.04506; .NET CLR 1.1.4322) Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.8.1.14) Gecko/20080404 Firefox/2.0.0.14 This sample code crashes FireFox: <BODY onload="go()"><SCRIPT> function go() { o=document.createElement("map"); document.body.parentNode.replaceChild(o,document.body); o.innerHTML='<param><legend><body></body><legend></legend></legend><style></style><input type=file></param>'; } </SCRIPT></BODY> Another example, appears to trigger the same bug: <BODY onload="go()"><SCRIPT> function go() { o=document.createElement('font'); document.body.appendChild(o); o.innerHTML='<fieldSet></fieldSet><frame>x</frame><style><input type=file>'; location.reload(); } </SCRIPT></BODY> Reproducible: Always Steps to Reproduce: Load one of these two URLs: * http://skypher.com/SkyLined/Repro/FireFox/AccessViolation%20cff39b5a%20(1).html * http://skypher.com/SkyLined/Repro/FireFox/AccessViolation%20cff39b5a%20(2).html Actual Results: FireFox crashes because of an Access Violation Marked as security problem because I'm not sure if it is a NULL pointer or not.
Yes, this appears to be a null deref. The first testcase crashes 2.0.0.14 but not trunk. I'll attach a backtrace from the branch crash.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Whiteboard: [sg:dos]
Attachment #326502 - Attachment mime type: application/octet-stream → text/plain
Product: Firefox → Core
QA Contact: general → general
I don't crash with trunk. Talkback ID: TB46895999M Stack Trace nsFileControlFrame::CreateAnonymousContent [mozilla/layout/forms/nsFileControlFrame.cpp, line 154] nsCSSFrameConstructor::CreateAnonymousFrames [mozilla/layout/base/nsCSSFrameConstructor.cpp, line 5885] nsCSSFrameConstructor::CreateAnonymousFrames [mozilla/layout/base/nsCSSFrameConstructor.cpp, line 5772] nsCSSFrameConstructor::ConstructHTMLFrame [mozilla/layout/base/nsCSSFrameConstructor.cpp, line 5706] nsCSSFrameConstructor::ConstructFrameInternal [mozilla/layout/base/nsCSSFrameConstructor.cpp, line 7910] nsCSSFrameConstructor::ConstructFrame [mozilla/layout/base/nsCSSFrameConstructor.cpp, line 7786] nsCSSFrameConstructor::ProcessInlineChildren [mozilla/layout/base/nsCSSFrameConstructor.cpp, line 13548] nsCSSFrameConstructor::ConstructInline [mozilla/layout/base/nsCSSFrameConstructor.cpp, line 13356] nsCSSFrameConstructor::ConstructFrameByDisplayType [mozilla/layout/base/nsCSSFrameConstructor.cpp, line 6813] nsCSSFrameConstructor::ConstructFrameInternal [mozilla/layout/base/nsCSSFrameConstructor.cpp, line 7974] nsCSSFrameConstructor::ConstructFrame [mozilla/layout/base/nsCSSFrameConstructor.cpp, line 7786] etc.. I get the same fix range as for bug 441561: http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2006-01-29+06&maxdate=2006-02-02+09&cvsroot=%2Fcvsroot So I guess this is probably related/the same as bug 441561.
Component: General → Layout
Keywords: crash, testcase
QA Contact: general → layout
Summary: DOM modification using JavaScript causes Access Violation (appears to be NULL pointer) → DOM modification using JavaScript causes Access Violation [@ nsFileControlFrame::CreateAnonymousContent] (appears to be NULL pointer)
Version: unspecified → 1.8 Branch
Group: core-security
Status: NEW → RESOLVED
Closed: 16 years ago
Flags: wanted1.9.0.x-
Flags: wanted1.8.1.x+
Resolution: --- → WORKSFORME
Crash Signature: [@ nsFileControlFrame::CreateAnonymousContent]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: