Closed Bug 1216521 Opened 10 years ago Closed 10 years ago

[CID 222168] leaked_storage: Variable rv 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)

Attachments

(1 file)

CID222168.patch
1.04 KB, patch
ekr
: review+
Details | Diff | Splinter Review
rv has to be freed here when returning NULL
Attached patch CID222168.patch — — Splinter Review
Attachment #8676230 - Flags: review?(martin.thomson)
Comment on attachment 8676230 [details] [diff] [review] CID222168.patch Review of attachment 8676230 [details] [diff] [review]: ----------------------------------------------------------------- Passing to ekr
Attachment #8676230 - Flags: review?(martin.thomson) → review?(ekr)
Comment on attachment 8676230 [details] [diff] [review] CID222168.patch Review of attachment 8676230 [details] [diff] [review]: ----------------------------------------------------------------- LGTM. Landed at: https://hg.mozilla.org/projects/nss/rev/4f05e85715d7
Attachment #8676230 - Flags: review?(ekr) → review+
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.21
Summary: [CID 222166] leaked_storage: Variable rv going out of scope leaks the storage it points to → [CID 222168] leaked_storage: Variable rv going out of scope leaks the storage it points to
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: