Closed Bug 45136 Opened 24 years ago Closed 24 years ago

Mail compose is leaking temp files

Categories

(MailNews Core :: Composition, defect, P3)

x86
Windows NT

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: phil, Assigned: rhp)

References

Details

(Whiteboard: [nsbeta3+])

Using 2000-07-10-08 commercial build on NT

I looked in my c:\temp directory, and there are ~100 files hanging around
labeled nsmail-NN.eml and nscopy-NN.tmp. They appear to be outgoing mail
messages. Of course, those should get cleaned up when the message is sent :-)
Hmm...I have a feeling that we are leaking these in failure situations (i.e. 
app crash, send failure, etc...) I know that if things get sent correctly, 
these are all cleaned up.

I'll throw it on the heap.

- rhp
Status: NEW → ASSIGNED
Target Milestone: --- → M18
Based on the files I see in the temp directory, we're leaking even when we don't
fail.
Grrrr...then something has possibly gone awry with RefCounting and the cleanup 
routine for the back end may not be getting called. I'll try to look into it.

- rhp
Keywords: correctness, nsbeta3
*** Bug 41641 has been marked as a duplicate of this bug. ***
Ok, I see what is going on here. I think I have a fix to make sure we clean 
these up.

- rhp
Summary: Mail compose is leaking temp files → [FIXED] Mail compose is leaking temp files
Whiteboard: [FIXED]
per selmer and mail/news PDT
Whiteboard: [FIXED] → [FIXED] [nsbeta3+]
Just checked in fix.

- rhp
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Summary: [FIXED] Mail compose is leaking temp files → Mail compose is leaking temp files
Whiteboard: [FIXED] [nsbeta3+] → [nsbeta3+]
QA Contact: lchiang → pmock
Verified as fixed on win32, linux, and macos using the following builds:
 win32 commercial seamonkey build 2000-091906-m18 installed on P500 Win98
 linux commercial seamonkey build 2000-091906-m18 installed on P200 RedHat 6.2
 macos commercial seamonkey build 2000-091904-m18 installed on G3/400 OS 9.04
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.