Crash displaying mail messages [@ nsGenericHTMLElement::FindAttributeImpact]

RESOLVED FIXED

Status

MailNews Core
Backend
--
critical
RESOLVED FIXED
15 years ago
10 years ago

People

(Reporter: Scott MacGregor, Assigned: Brian Ryner (not reading))

Tracking

({crash})

Trunk
x86
Windows XP
crash

Firefox Tracking Flags

(Not tracked)

Details

(crash signature)

Attachments

(1 attachment)

(Reporter)

Description

15 years ago
I just pulled a tree picking up the static atom changes in mozilla\html\content
(after the most recent changes late this afternoon). 

I'm crashing when trying to view various messages in:

nsGenericHTMLElement::FindAttributeImpact
if (aAttribute == *map->attribute) {

looks like the value of map is pointing to invalid memory. 

Alec do you think this is related to the static atom changes in content\html or
is it something else?
(Reporter)

Comment 1

15 years ago
I should have added that the caller of FindAttributeImpact is:

nsHTMLFontElement::GetMappedAttributeImpact
(Reporter)

Comment 2

15 years ago
Created attachment 120774 [details] [diff] [review]
fix for the crash
(Reporter)

Comment 3

15 years ago
I think Bryner may have already checked something in similar to my patch to fix
this crash.

Updated

15 years ago
Severity: normal → critical
Keywords: crash
Summary: Crash displaying mail messages [nsGenericHTMLElement::FindAttributeImpact] → Crash displaying mail messages [@ nsGenericHTMLElement::FindAttributeImpact]

Comment 4

15 years ago
yep, sorry my bad - I had actually fixed this on one machine, and then checked
in from another.. doh!
over to bryner (I'll mark it fixed too)
Assignee: alecf → bryner

Comment 5

15 years ago
and now marking it fixed.
Status: NEW → RESOLVED
Last Resolved: 15 years ago
Resolution: --- → FIXED
Product: MailNews → Core
Product: Core → MailNews Core
Crash Signature: [@ nsGenericHTMLElement::FindAttributeImpact]
You need to log in before you can comment on or make changes to this bug.