428830 - Thunderbird startup leaks nsLocalFile due to bug 388833

Status: RESOLVED FIXED

(Thunderbird :: Build Config, defect)

Description

[Mark Banner (:standard8)]

Attachment: The fix

Thunderbird leaks an nsLocaleFile (and conceivably its nsStringBuffer), probably due to bug 388833. According to my standalone-tinderbox I have been running we get these leaks on the Mail RLk test:

TOTAL                                           384       0.00%
nsLocalFile                                     360       0.00%
nsStringBuffer                                   24       0.00%

I picked the fix up after doing a bit of research and finding bug 389779 - basically we need to add a bit of code to nsMailApp.cpp to do things cleanly.

See attached patch, with this patch in, and the current form of the tinderbox tests, Mail RLk goes down to 0B for TB.

This probably accounts for some, but not all of the leaks in bug 426963 (yes we probably need to expand the tinderbox tests a bit more).

Comment 1

[Benjamin Smedberg]

Yeah, it's a fictional leak, but since clean leak logs are good the extra loginit/logterm pair will get you where you want to be.

r=me, FWIW, though a mailnews peer should still give MOA.

Comment 2

[Mark Banner (:standard8)]

Comment on attachment 315401 [details] [diff] [review]
The fix

Moving review as dmose hasn't got privs in mail/ yet

Comment 3

[Phil Ringnalda (:philor)]

Comment on attachment 315401 [details] [diff] [review]
The fix

After careful study and long analysis, I believe what bsmedberg tells me ;)

r=philringnalda

Comment 4

[Mark Banner (:standard8)]

Patch checked in -> fixed.