Closed
Bug 1305745
Opened 8 years ago
Closed 8 years ago
Memory leak in ElfLoader::GetMappableFromPath
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox52 fixed)
RESOLVED
FIXED
Firefox 52
| Tracking | Status | |
|---|---|---|
| firefox52 | --- | fixed |
People
(Reporter: esawin, Assigned: esawin)
Details
Attachments
(1 file)
|
675 bytes,
patch
|
glandium
:
review+
|
Details | Diff | Splinter Review |
We're leaking zip_path, which is duplicated in ElfLoader::GetMappableFromPath, passed to ZipCollection::GetZip where it's duplicated (and freed on destruction) again.
|
Assignee | |
Comment 1•8 years ago
|
||
Free the zip path.
Attachment #8795362 -
Flags: review?(mh+mozilla)
|
||
Updated•8 years ago
|
Attachment #8795362 -
Flags: review?(mh+mozilla) → review+
Pushed by esawin@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/52982dbd3d40 [1.1] Free the zip path intermediate variable. r=glandium
|
||
Comment 3•8 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/52982dbd3d40
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox52:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
|
||
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•