ASSERTION: ReleaseSubtree not called with xml parsing error in attribute

VERIFIED FIXED in mozilla1.9beta2

Status

()

P2
normal
VERIFIED FIXED
11 years ago
11 years ago

People

(Reporter: martijn.martijn, Assigned: peterv)

Tracking

({assertion, memory-leak, testcase})

Trunk
mozilla1.9beta2
assertion, memory-leak, testcase
Points:
---
Bug Flags:
blocking1.9 +
in-testsuite ?

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

11 years ago
Created attachment 287667 [details]
testcase

(I have no idea where to put this in)

See testcase, which triggers this assertion when loading it in current debug trunk build on windows:
###!!! ASSERTION: ReleaseSubtree not called: '!mChildren && mNumChildren == 0',
file c:\mozilla-build\mozilla\content\xul\content\src\nsXULElement.h, line 259

When looking at the source of that:
http://lxr.mozilla.org/seamonkey/source/content/xul/content/src/nsXULElement.h#255
It seems to me that this could be a memory leak.
(Assignee)

Comment 1

11 years ago
Yes, this leaks the prototype elements.
Assignee: nobody → peterv
Flags: blocking1.9?
OS: Windows XP → All
Priority: -- → P2
Hardware: PC → All
Target Milestone: --- → mozilla1.9 M10
(Assignee)

Comment 2

11 years ago
Created attachment 287707 [details] [diff] [review]
v1
Attachment #287707 - Flags: superreview?(jonas)
Attachment #287707 - Flags: review?(jonas)
Attachment #287707 - Flags: superreview?(jonas)
Attachment #287707 - Flags: superreview+
Attachment #287707 - Flags: review?(jonas)
Attachment #287707 - Flags: review+

Updated

11 years ago
Flags: blocking1.9? → blocking1.9+
(Assignee)

Updated

11 years ago
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
(Reporter)

Comment 3

11 years ago
Verified fixed after updating my debug trunk build to today.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.