Closed Bug 1214829 Opened 10 years ago Closed 10 years ago

[CID 749513] leaked_storage: Variable filename going out of scope leaks the storage it points to

Categories

(NSS :: Libraries, defect)

Product:
NSS
Component:
Libraries
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox44 affected)

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox44 --- affected

People

(Reporter: franziskus, Assigned: franziskus)

References

(Blocks 1 open bug)

Details

(Keywords: coverity)

filename variable has to be freed if secmod is null.
Patch is at https://codereview.appspot.com/269310044/
Flags: needinfo?(martin.thomson)
Flags: needinfo?(ekr)
goto is the answer.
Flags: needinfo?(martin.thomson)
Flags: needinfo?(ekr)
new version now with goto loser (freeing everything if there) https://codereview.appspot.com/269310044/
Flags: needinfo?(martin.thomson)
https://hg.mozilla.org/projects/nss/rev/498d0adca5e7
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: needinfo?(martin.thomson)
OS: Unspecified → All
Hardware: Unspecified → All
Resolution: --- → FIXED
Target Milestone: --- → 3.21
You need to log in before you can comment on or make changes to this bug.