Closed
Bug 337332
Opened 19 years ago
Closed 19 years ago
Coverity Leak in OpenNewDB (security/nss/lib/softoken/keydb.c)
Categories
(NSS :: Libraries, defect, P2)
NSS
Libraries
Tracking
(Not tracked)
RESOLVED
FIXED
3.11.2
People
(Reporter: kherron+mozilla, Assigned: alvolkov.bgs)
References
()
Details
(Keywords: coverity, Whiteboard: [CID 580] FIPS)
Attachments
(1 file)
|
987 bytes,
patch
|
nelson
:
review+
|
Details | Diff | Splinter Review |
Please refer to the sample URL. At line 945, |nsslowkey_NewHandle| is called which allocates a handle. The same function is called again at line 954, overwriting the pointer to the first allocation. It looks like the first call is just a mistake.
|
Reporter | |
Updated•19 years ago
|
Whiteboard: [good first bug]
|
|
||
Updated•19 years ago
|
Target Milestone: --- → 3.11.2
|
|
||
Updated•19 years ago
|
Priority: -- → P2
|
|
Assignee | |
Updated•19 years ago
|
Assignee: nobody → alexei.volkov.bugs
Whiteboard: [good first bug] → [good first bug] PKIX
|
|
Assignee | |
Updated•19 years ago
|
Whiteboard: [good first bug] PKIX → [good first bug] FIPS
|
|
Assignee | |
Comment 1•19 years ago
|
||
Attachment #222286 -
Flags: review?(nelson)
|
|
||
Comment 2•19 years ago
|
||
Comment on attachment 222286 [details] [diff] [review]
remove first init of "updateHandle"
r=nelson
Attachment #222286 -
Flags: review?(nelson) → review+
|
|
Assignee | |
Comment 3•19 years ago
|
||
trunk
/cvsroot/mozilla/security/nss/lib/softoken/keydb.c,v <-- keydb.c
new revision: 1.45; previous revision: 1.44
3.11 branch
/cvsroot/mozilla/security/nss/lib/softoken/keydb.c,v <-- keydb.c
new revision: 1.40.2.3; previous revision: 1.40.2.2
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
|
|
||
Updated•19 years ago
|
Summary: Leak in OpenNewDB (security/nss/lib/softoken/keydb.c) → Coverity Leak in OpenNewDB (security/nss/lib/softoken/keydb.c)
You need to log in
before you can comment on or make changes to this bug.
Description
•