During a purify run of a test tool of mine, I discovered that PK11_WrapPrivKey() leaks a little memory. Specifically, 'param' is never freed.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Created attachment 99897 [details] [diff] [review] Free params if we allocated them Good catch. The code already was set up to handle the case where it needed to free the params, it just didn't get around to doing it.
Setting target milestone. wtc, could I have a review of this. bob
Target Milestone: --- → 3.6
Comment on attachment 99897 [details] [diff] [review] Free params if we allocated them r=wtc. Bob, your patch is correct.
Attachment #99897 - Flags: review+
Fix checked in.
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.