RDFResource and RDFServiceImpl leak on startup test

RESOLVED FIXED

Status

defect
RESOLVED FIXED
12 years ago
11 years ago

People

(Reporter: mscott, Assigned: mscott)

Tracking

({fixed1.8.1.8, memory-leak})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

12 years ago
Posted 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?

Updated

12 years ago
Attachment #272269 - Flags: superreview?(bienvenu) → superreview+
(Assignee)

Updated

12 years ago
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED

Updated

12 years ago
Keywords: mlk

Comment 1

12 years ago
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+
(Assignee)

Updated

12 years ago
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.