Closed
Bug 16361
Opened 25 years ago
Closed 25 years ago
[DOGFOOD][MLK] nsEditor::CreateNode() and nsEditor::RemoveAttribute() leak
Categories
(Core :: DOM: Editor, defect, P1)
Core
DOM: Editor
Tracking
()
VERIFIED
FIXED
M11
People
(Reporter: mozilla, Assigned: mozeditor)
References
()
Details
(Whiteboard: [PDT+]MAJOR LEAK)
Lots of function in nsEditor, such as:
nsEditor::CreateNode() [around line #893]
nsEditor::SetAttribute() [around line #846]
nsEditor::RemoveAttribute() [around line #879]
(and perhaps more, I didn't check them all)
appear to potentially leak a ChangeAttributeTxn. Please fix these leaks...
either add NS_RELEASE()s for each ChangeAttributeTxn or switch to using
nsCOMPtrs, as appropriate.
[Note: leak found using beard's Boehm GC]
Reporter | ||
Updated•25 years ago
|
Priority: P3 → P2
Target Milestone: M11
Updated•25 years ago
|
Assignee: sfraser → jfrancis
Assignee | ||
Comment 1•25 years ago
|
||
accepting/m11
Updated•25 years ago
|
Target Milestone: M11 → M13
Updated•25 years ago
|
Severity: normal → critical
Priority: P2 → P1
Summary: [MLK] nsEditor::CreateNode() and nsEditor::RemoveAttribute() leak → [DOGFOOD][MLK] nsEditor::CreateNode() and nsEditor::RemoveAttribute() leak
Whiteboard: MAJOR LEAK
Target Milestone: M13 → M11
Comment 2•25 years ago
|
||
this is a major leak and we know the cause of the leak. So I am moving it back
to M11, setting priority to P1 and making it a dogfood candidate.
Assignee | ||
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•25 years ago
|
||
fixed
Assignee | ||
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 7•25 years ago
|
||
i have verified that the editor no longer leaks transactions.
You need to log in
before you can comment on or make changes to this bug.
Description
•