Closed Bug 385716 Opened 17 years ago Closed 17 years ago

inIFlasher.drawElementOutline and inIFlasher.repaintElement aren't null safe [@ inLayoutUtils::GetWindowFor]

Categories

(Core :: Layout, defect)

x86
Windows XP
defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla1.9alpha8

People

(Reporter: timeless, Assigned: timeless)

Details

(Keywords: crash)

Crash Data

Attachments

(1 file)

i was trying to use it from js and accidentally passed null.

firefox!inLayoutUtils::GetWindowFor(class nsIDOMNode * aNode = 0x00000000)+0x14 [e:\builds\tinderbox\fx-trunk\winnt_5.2_depend\mozilla\layout\inspector\src\inlayoututils.cpp @ 60]
firefox!inFlasher::DrawElementOutline(class nsIDOMElement * aElement = 0x00000000)+0xf [e:\builds\tinderbox\fx-trunk\winnt_5.2_depend\mozilla\layout\inspector\src\inflasher.cpp @ 152]
xpcom_core!NS_InvokeByIndex_P(class nsISupports * that = 0x01908a78, unsigned int methodIndex = 9, unsigned int paramCount = 1, struct nsXPTCVariant * params = 0x0012d340)+0x27
Assignee: nobody → timeless
Status: NEW → ASSIGNED
Attachment #269808 - Flags: superreview?(bzbarsky)
Attachment #269808 - Flags: review?(bzbarsky)
Comment on attachment 269808 [details] [diff] [review]
ensure arg pointer

Sure
Attachment #269808 - Flags: superreview?(bzbarsky)
Attachment #269808 - Flags: superreview+
Attachment #269808 - Flags: review?(bzbarsky)
Attachment #269808 - Flags: review+
Comment on attachment 269808 [details] [diff] [review]
ensure arg pointer

mozilla/layout/inspector/src/inFlasher.cpp 	1.35
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Flags: in-testsuite?
Target Milestone: --- → mozilla1.9beta1
Crash Signature: [@ inLayoutUtils::GetWindowFor]
Product: Core → Core Graveyard
Component: Layout: Misc Code → Layout
Product: Core Graveyard → Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: