Closed Bug 15696 Opened 25 years ago Closed 25 years ago

variable "atom" is a leak

Categories

(Core :: DOM: Editor, defect, P3)

Product:
Core
Component:
DOM: Editor
Platform:
x86
Windows NT
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: buster, Assigned: mozeditor)

Details

in PlaceholderTxn::Merge(), "atom" is initialized using GetTxnName which addrefs, but there is no release.
Assignee: jfrancis → buster
I think buster has a fix for this ready to go in. Assigning to him. Reassign back to me if this isn't correct, or if you want me to do the checkin.
no, I fixed some other leaks and noticed this one. Please take care of it at your leisure. I was holding too much in my tree and badly needed to check in today, so I didn't have time for it even though it's pretty easy.
Assignee: buster → jfrancis
forgot to reassign
accepting/m11
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
fixed
joe, remind me which fix this was. which module? what line?
thanks buster! the fix is in PlaceHolderTxn.cpp, line 176.
Great now buster can help me mark this puppy verified-fixed....
Can we mark this verified-fixed ???
Status: RESOLVED → REOPENED
Target Milestone: M11 → M12
I'm going to reopen and m12 this bug. There is a problem with my fix that can still cause a leak in some very rare circumstances. I'll fix it correctly sometime soon (tree opening for m12 sometime soon).
Resolution: FIXED → ---
Clearing FIXED resolution due to reopen of this bug.
Status: REOPENED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → FIXED
fixed for real this time
Joe ,can you mark this verified again? thanks!
Status: RESOLVED → VERIFIED
verify one my mark: ... ... ...Mark!
You need to log in before you can comment on or make changes to this bug.