Closed
Bug 103779
Opened 24 years ago
Closed 24 years ago
GetWindowsFolder leaks a string
Categories
(Core :: XPCOM, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: Bienvenu, Assigned: Bienvenu)
References
Details
(Keywords: memory-leak)
Attachments
(1 file)
|
545 bytes,
patch
|
dougt
:
review+
sspitzer
:
superreview+
|
Details | Diff | Splinter Review |
GetWindowsFolder in nsSpecialSystemDirectory leaks a string by duping it and
assigning the dup to an nsFileSpec &, which leaks the dup, since nsFileSpec
copies the data anyway. Patch upcoming.
|
Assignee | |
Comment 2•24 years ago
|
||
|
|
||
Comment 3•24 years ago
|
||
Comment on attachment 52659 [details] [diff] [review]
proposed fix
sr=sspitzer
Attachment #52659 -
Flags: superreview+
|
||
Comment 4•24 years ago
|
||
Comment on attachment 52659 [details] [diff] [review]
proposed fix
looks good.
Attachment #52659 -
Flags: review+
|
|
Assignee | |
Comment 6•24 years ago
|
||
fixed.
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
How big was this leak? I filed bug 102828, which is very similar.
|
|
Assignee | |
Comment 8•24 years ago
|
||
*** Bug 102828 has been marked as a duplicate of this bug. ***
Verified FIXED with the latest CVS pull and Purify, with the following scenario
from bug 102828 (which I filed).
* Launching mozilla.exe -mail with 1 POP3 server, with 2 new messages.
* Login at startup and saved password selected.
* Didn't actually read mail, just let biff notify me of new messages
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•