Closed
Bug 388105
Opened 17 years ago
Closed 17 years ago
RDFResource and RDFServiceImpl leak on startup test
Categories
(MailNews Core :: Backend, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mscott, Assigned: mscott)
Details
(Keywords: fixed1.8.1.8, memory-leak)
Attachments
(1 file)
948 bytes,
patch
|
Bienvenu
:
superreview+
dveditz
:
approval1.8.1.8+
|
Details | Diff | Splinter 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•17 years ago
|
Attachment #272269 -
Flags: superreview?(bienvenu) → superreview+
Assignee | ||
Updated•17 years ago
|
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Comment 1•17 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 2•17 years ago
|
||
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•17 years ago
|
Keywords: fixed1.8.1.7
Updated•16 years ago
|
Product: Core → MailNews Core
You need to log in
before you can comment on or make changes to this bug.
Description
•