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)

3.11.7

Tracking

(Not tracked)

RESOLVED FIXED
3.15.5

People

(Reporter: wtc, Assigned: wtc)

Details

Attachments

(1 file)

Attached patch PatchSplinter 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 on attachment 8347558 [details] [diff] [review]
Patch

Review of attachment 8347558 [details] [diff] [review]:
-----------------------------------------------------------------

r+ ryan.sleevi
Attachment #8347558 - Flags: review?(ryan.sleevi) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: 3.15.4 → 3.15.5
Target Milestone: 3.15.5 → 3.16
Target Milestone: 3.16 → 3.15.5
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: