Closed Bug 695138 Opened 8 years ago Closed 8 years ago

Fix inout memory leak in XPConnect tests

Categories

(Core :: XPConnect, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla10

People

(Reporter: bholley, Assigned: bholley)

Details

Attachments

(1 file)

I was unclear as to the memory ownership model for inout parameters with dependent buffers, so I got it wrong in nsXPCTestParams::TestString and friends.

I talked to bsmedberg, and clarified the situation. I've updated the docs here to reflect them:
https://developer.mozilla.org/en/XPIDL

The test still needs fixing though.
Attached a fix. Flagging khuey for review.


Review: khuey
Comment on attachment 568305 [details] [diff] [review]
Fix inout memory leak in XPConnect tests. v1

Oops, didn't actually flag. Flagging khuey.
Attachment #568305 - Flags: review?(khuey)
Pushed to mozilla-inbound:
http://hg.mozilla.org/integration/mozilla-inbound/rev/a2d4d5fd846a
Target Milestone: --- → mozilla10
https://hg.mozilla.org/mozilla-central/rev/a2d4d5fd846a
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.