Closed
Bug 950299
Opened 10 years ago
Closed 10 years ago
ocsp_CertIDsMatch should return PR_FALSE instead of SECFailure on error
Categories
(NSS :: Libraries, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
3.15.5
People
(Reporter: wtc, Assigned: wtc)
Details
Attachments
(1 file)
1.74 KB,
patch
|
ryan.sleevi
:
review+
wtc
:
checked-in+
|
Details | Diff | Splinter Review |
This bug was introduced in the patch for bug 338986. ocsp_CertIDsMatch returns PRBool. So if it encounters an error, it has to return PR_FALSE rather than SECFailure. I also removed the unused parameter |handle|.
Attachment #8347558 -
Flags: review?(ryan.sleevi)
Comment 1•10 years ago
|
||
Comment on attachment 8347558 [details] [diff] [review] Patch Review of attachment 8347558 [details] [diff] [review]: ----------------------------------------------------------------- r+ ryan.sleevi
Attachment #8347558 -
Flags: review?(ryan.sleevi) → review+
Assignee | ||
Comment 2•10 years ago
|
||
Comment on attachment 8347558 [details] [diff] [review] Patch Patch checked in: https://hg.mozilla.org/projects/nss/rev/1b97dd5ae003
Attachment #8347558 -
Flags: checked-in+
Assignee | ||
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: 3.15.4 → 3.15.5
Updated•10 years ago
|
Target Milestone: 3.15.5 → 3.16
Assignee | ||
Updated•10 years ago
|
Target Milestone: 3.16 → 3.15.5
You need to log in
before you can comment on or make changes to this bug.
Description
•