Created attachment 814632 [details] [diff] [review] Patch tstclnt.c has a local variable that is poorly named: PRErrorCode status; because |status| is conventionally used with the PRStatus or SECStatus type. The variable should be renamed |error|, which is also the name of the argument for SSL_AuthCertificateComplete.
Attachment #814632 - Flags: review?(brian)
Attachment #814632 - Flags: review?(brian) → review+
Patch checked in: https://hg.mozilla.org/projects/nss/rev/c623bea95f82
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
changing target milestone to 3.15.4
Target Milestone: 3.15.3 → 3.15.4
You need to log in before you can comment on or make changes to this bug.