Closed Bug 388105 Opened 14 years ago Closed 14 years ago

RDFResource and RDFServiceImpl leak on startup test

Categories

(MailNews Core :: Backend, defect)

x86
Windows Vista
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mscott, Assigned: mscott)

Details

(Keywords: fixed1.8.1.8, memory-leak)

Attachments

(1 file)

Attached patch the fixSplinter Review
We leak 3 RDF resources and the RDF service when doing a startup / shutdown test. 

this low risk leak fix might be something we want on the branch too.
Attachment #272269 - Flags: superreview?(bienvenu)
Attachment #272269 - Flags: approval1.8.1.6?
Attachment #272269 - Flags: superreview?(bienvenu) → superreview+
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Keywords: mlk
I wonder if the shutdown crash I saw in bug 388241 has to do something with this. mailnews appears way down in the stacktrace and this patch here is the only recent checkin that mentions RDF datasources. The actual crash there seems to be happening when freeing a xul template RDF node.
Comment on attachment 272269 [details] [diff] [review]
the fix

approved for 1.8.1.7, a=dveditz for release-drivers
Attachment #272269 - Flags: approval1.8.1.7? → approval1.8.1.7+
Keywords: fixed1.8.1.7
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.