update/tests/TestAUSHelper.cpp CheckMsg leaks |inFP| and |mbuf|

RESOLVED FIXED in Firefox 39

Status

()

Toolkit
Application Update
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: erahm, Assigned: rstrong)

Tracking

({coverity})

Trunk
mozilla39
x86_64
Linux
coverity
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox39 fixed)

Details

(Whiteboard: [CID 1286183][CID 1286097])

Attachments

(1 attachment)

Note: this is in test code.

CheckMsg leaks |inFP| [1] in various early exits.

[1] https://hg.mozilla.org/mozilla-central/annotate/0190a1d17294/toolkit/mozapps/update/tests/TestAUSHelper.cpp
Eric, this is a binary only used by tests and hence won't help the memshrink goals as I understand them from the memshrink page.
"MemShrink is a project that aims to reduce the memory consumption of Firefox (on desktop and mobile) and Firefox OS."
https://wiki.mozilla.org/Performance/MemShrink

I'm fine with this being fixed but this code is in no way tied to any application that we ship.

Can you provide details how this helps memshrink? Thanks!
Flags: needinfo?(erahm)
Summary: TestAUSHelper.cpp CheckMsg leaks |inFP| → update/tests/TestAUSHelper.cpp CheckMsg leaks |inFP|
(In reply to Robert Strong [:rstrong] (use needinfo to contact me) from comment #1)
> Can you provide details how this helps memshrink? Thanks!

I'm currently doing a coverity triage, I'm flagging anything showing up as a resource leak is being memshrink. Generally we'd make the decision whether or not a memshrink tag is appropriate in the bi-weekly memshrink triage.

But in this case it sounds like it's fine to clear.
Flags: needinfo?(erahm)
Whiteboard: [MemShrink][CID 1286183] → [CID 1286183]
It looks like |mbuf| [1] is also leaked.

[1] https://hg.mozilla.org/mozilla-central/annotate/0190a1d17294/toolkit/mozapps/update/tests/TestAUSHelper.cpp#l105
Summary: update/tests/TestAUSHelper.cpp CheckMsg leaks |inFP| → update/tests/TestAUSHelper.cpp CheckMsg leaks |inFP| and |mbuf|
Whiteboard: [CID 1286183] → [CID 1286183][CID 1286097]
Created attachment 8577672 [details] [diff] [review]
patch
Assignee: nobody → robert.strong.bugs
Status: NEW → ASSIGNED
Attachment #8577672 - Flags: review?(netzen)
Attachment #8577672 - Flags: review?(netzen) → review+
Pushed to fx-team
https://hg.mozilla.org/integration/fx-team/rev/db07f0b48a2d
Flags: in-testsuite+
Target Milestone: --- → mozilla39
https://hg.mozilla.org/mozilla-central/rev/db07f0b48a2d
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox39: --- → fixed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.