RDFResource and RDFServiceImpl leak on startup test

RESOLVED FIXED

Status

MailNews Core
Backend
RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: Scott MacGregor, Assigned: Scott MacGregor)

Tracking

({fixed1.8.1.8, mlk})

Trunk
x86
Windows Vista
fixed1.8.1.8, mlk

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
Created attachment 272269 [details] [diff] [review]
the fix

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

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

Updated

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

Updated

10 years ago
Keywords: mlk

Comment 1

10 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

10 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.