crash in nsDOMWindowUtils::GetFileId with nsDOMWindowUtils::getFileId(null)

RESOLVED FIXED in mozilla15

Status

()

Core
DOM
--
critical
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Martijn Wargers (dead), Assigned: janv)

Tracking

({crash, testcase})

Trunk
mozilla15
x86
Windows NT
crash, testcase
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(crash signature)

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Created attachment 624528 [details]
testcase (uses enhanced privileges)

This bug was filed from the Socorro interface and is 
report bp-1c26e597-d903-4787-b96a-f53aa2120516 .
============================================================= 
0 	xul.dll 	nsDOMWindowUtils::GetFileId 	dom/base/nsDOMWindowUtils.cpp:2275
1 	xul.dll 	NS_InvokeByIndex_P 	xpcom/reflect/xptcall/src/md/win32/xptcinvoke.cpp:102
2 	xul.dll 	XPCWrappedNative::CallMethod 	js/xpconnect/src/XPCWrappedNative.cpp:2389
3 	xul.dll 	XPC_WN_CallMethod 	js/xpconnect/src/XPCWrappedNativeJSOps.cpp:1541
4 	mozjs.dll 	js::types::TypeSet::addType 	js/src/jsinferinlines.h:1158
5 	mozjs.dll 	js::Interpret 	js/src/jsinterp.cpp:2744
6 	mozjs.dll 	js::types::TypeScript::SetThis 	js/src/jsinferinlines.h:723
7 	mozjs.dll 	js::ExecuteKernel 	js/src/jsinterp.cpp:678
8 	mozjs.dll 	js::Execute 	js/src/jsinterp.cpp:720
9 	mozjs.dll 	EvaluateUCScriptForPrincipalsCommon 	js/src/jsapi.cpp:5321
10 	mozjs.dll 	JS_EvaluateUCScriptForPrincipalsVersionOrigin 	js/src/jsapi.cpp:5358
11 	xul.dll 	nsJSContext::EvaluateString 	dom/base/nsJSEnvironment.cpp:1485
12 	xul.dll 	nsScriptLoader::EvaluateScript 	content/base/src/nsScriptLoader.cpp:910
13 	xul.dll 	nsScriptLoader::ProcessRequest 	content/base/src/nsScriptLoader.cpp:803
14 	xul.dll 	nsScriptLoader::ProcessScriptElement 	content/base/src/nsScriptLoader.cpp:749
15 	xul.dll 	nsScriptElement::MaybeProcessScript 	content/base/src/nsScriptElement.cpp:169

Comment 1

5 years ago
I don't crash with the testcase.
(Reporter)

Comment 2

5 years ago
Because it uses enhanced privileges, it's probably easiest in current builds to test it from chrome:// urls, since they removed the enhanced privs UI.
Jan, could you take a look?
Blocks: 661877
(Assignee)

Comment 4

5 years ago
ok
Assignee: nobody → Jan.Varga
Status: NEW → ASSIGNED
(Assignee)

Comment 5

5 years ago
the patch for 726593 fixes this crash, I changed getFileId() to take a jsval
anyway, I added a new test to check the getFileId(null)
Depends on: 726593
(Assignee)

Comment 6

5 years ago
fixed along with bug 726593
(Assignee)

Updated

5 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Updated

5 years ago
Target Milestone: --- → mozilla15
You need to log in before you can comment on or make changes to this bug.