Closed Bug 405639 Opened 15 years ago Closed 15 years ago

Dynamically created <svg:use> reference loop causes nsGenericElement leak

Categories

(Core :: SVG, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla1.9beta2

People

(Reporter: jruderman, Assigned: peterv)

References

Details

(Keywords: memory-leak, testcase)

Attachments

(2 files)

Attached image testcase
Steps to reproduce:
1. export XPCOM_MEM_LEAK_LOG=2
2. Run a debug build of Firefox.
3. Load the testcase.
4. Quit Firefox.

Result: trace-refcnt reports that several nsGenericElement objects (along with a bunch of others) have leaked.

Bug 376666 might be related.
Attached patch v1Splinter Review
Unlinking seems safe, mOriginal/mClone/mSourceContent are null-checked before use.
Assignee: nobody → peterv
Status: NEW → ASSIGNED
Attachment #290594 - Flags: superreview?(jonas)
Attachment #290594 - Flags: review?(jonas)
Attachment #290594 - Flags: superreview?(jonas)
Attachment #290594 - Flags: superreview+
Attachment #290594 - Flags: review?(jonas)
Attachment #290594 - Flags: review+
Attachment #290594 - Flags: approval1.9+
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Flags: in-testsuite?
OS: Mac OS X → All
Priority: -- → P3
Hardware: PC → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9 M10
Depends on: 406361
Crashtest checked in.
Flags: in-testsuite? → in-testsuite+
You need to log in before you can comment on or make changes to this bug.