Closed
Bug 337345
Opened 20 years ago
Closed 20 years ago
Leak in nsCrypto::GenerateCRMFRequest (security/manager/ssl/src/nsCrypto.cpp)
Categories
(Core Graveyard :: Security: UI, defect)
Core Graveyard
Security: UI
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kherron+mozilla, Assigned: KaiE)
References
()
Details
(Keywords: coverity, fixed1.8.1, Whiteboard: [good first bug])
Attachments
(1 file)
|
708 bytes,
patch
|
rrelyea
:
review+
darin.moz
:
approval1.8.1+
|
Details | Diff | Splinter Review |
This is coverity issue 576. Please refer to the sample URL. At lines 1584-1586, the |keyids| array allocated at line 1552 is leaked.
|
Reporter | |
Updated•20 years ago
|
Whiteboard: [good first bug]
|
Assignee | |
Comment 1•20 years ago
|
||
Attachment #221503 -
Flags: review?(rrelyea)
|
||
Comment 2•20 years ago
|
||
Comment on attachment 221503 [details] [diff] [review]
Patch v1
r+ = rrelyea
Attachment #221503 -
Flags: review?(rrelyea) → review+
|
|
Assignee | |
Comment 3•20 years ago
|
||
fixed on trunk
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 4•20 years ago
|
||
Comment on attachment 221503 [details] [diff] [review]
Patch v1
should this leak fix land on the 1.8 branch?
Attachment #221503 -
Flags: approval1.8.1?
|
|
||
Comment 5•20 years ago
|
||
Comment on attachment 221503 [details] [diff] [review]
Patch v1
a=darin on behalf of drivers
Attachment #221503 -
Flags: approval1.8.1? → approval1.8.1+
|
||
Updated•9 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•