Closed Bug 87960 Opened 23 years ago Closed 23 years ago

leaks searching messages

Categories

(MailNews Core :: Backend, defect)

x86
Windows 2000
defect
Not set
normal

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
Keywords: mlk
Attached file a leak log for your enjoyment (deleted) —
-> putterman I'm available to help/answer questions.
Assignee: dprice → putterman
Target Milestone: --- → mozilla0.9.3
reassigning to naving
Assignee: putterman → naving
Target Milestone: mozilla0.9.3 → mozilla0.9.4
boy it would be good if we could nail some of this mail leaks in the next week.
Jesus christ, 850k is huge. I agree that it would be great if this was high priority. (BTW, isn't this a topmlk?)
Blocks: 91387
Attached patch proposed fix (deleted) — Splinter Review
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.
Attached file leak log after the fix (deleted) —
david, please review.
r=bienvenu - please don't forget to get an sr before checking in.
sr=sspitzer, make sure to test well.
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
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: