Closed
Bug 219756
Opened 22 years ago
Closed 22 years ago
A leaked CERTCertificate reference in CryptoManager.c:ConfigureOCSP.
Categories
(JSS Graveyard :: Library, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
3.4.2
People
(Reporter: wtc, Assigned: wtc)
Details
Attachments
(1 file)
|
1.75 KB,
patch
|
Details | Diff | Splinter Review |
In CryptoManager.c, the function ConfigureOCSP calls
CERT_FindCertByNickname or PK11_FindCertByNickname,
but the returned CERTCertificate reference is leaked.
|
Assignee | |
Comment 1•22 years ago
|
||
Destroy the CERTCertificate reference before leaving
the lexical scope in which it is used.
Note that the indentation of this code is messed up,
so it is not easy to see what the lexical scope is.
|
|
Assignee | |
Comment 2•22 years ago
|
||
Patch checked in on the JSS trunk (JSS 3.4.2). Glen, you
are welcome to review the patch.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.4.2
|
||
Comment 3•22 years ago
|
||
Thanks Wan-Teh. Review patch is good, and I added the patch to the Sun Orion
branch.
You need to log in
before you can comment on or make changes to this bug.
Description
•