Closed
Bug 87960
Opened 23 years ago
Closed 23 years ago
leaks searching messages
Categories
(MailNews Core :: Backend, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9.4
People
(Reporter: dprice, Assigned: naving)
References
Details
(Keywords: memory-leak)
Attachments
(4 files)
set XPCOM_MEM_LEAK_LOG=filename
start mozilla -mail
read messages in a newsgroup
open search mail/news messages
subject contains "foo"
search
exit
check out the leak log. It indicates we've leaked about 850K
for this test I used a mail profile with a news server, news.mozilla.org, I read
messages in n.p.m.porkjockeys
Reporter | ||
Comment 1•23 years ago
|
||
Reporter | ||
Comment 2•23 years ago
|
||
-> putterman
I'm available to help/answer questions.
Assignee: dprice → putterman
Target Milestone: --- → mozilla0.9.3
Comment 3•23 years ago
|
||
reassigning to naving
Assignee: putterman → naving
Target Milestone: mozilla0.9.3 → mozilla0.9.4
Comment 4•23 years ago
|
||
boy it would be good if we could nail some of this mail leaks in the next week.
Comment 5•23 years ago
|
||
Jesus christ, 850k is huge. I agree that it would be great if this was high
priority. (BTW, isn't this a topmlk?)
Assignee | ||
Comment 6•23 years ago
|
||
Assignee | ||
Comment 7•23 years ago
|
||
Ok, so have a fix for this leak. The problem was that scopeTerm was holding
a strong reference to msgSearchSession so SearchSession was not being free'd
I will attach the leak log after the fix.
Assignee | ||
Comment 8•23 years ago
|
||
Assignee | ||
Comment 9•23 years ago
|
||
david, please review.
Comment 10•23 years ago
|
||
r=bienvenu - please don't forget to get an sr before checking in.
Comment 11•23 years ago
|
||
sr=sspitzer, make sure to test well.
Assignee | ||
Comment 12•23 years ago
|
||
fix checked in.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
QA Contact: esther → stephend
Verified FIXED using build 2001-08-15-04 on Windows 2000, using the same
criteria as above. Note: news search still leaks, just in other places (already
filed).
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: MailNews → Core
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
•