Skip to content

Commit b8f1b44

Browse files
committed
Enable certification verification using CRL-DP
Currently certificates can be validated only using OCSP with a configured responder or using the AIA certificate extension. If the responder cannot be used verification is not possible. This is the case for the startup certificates of the responder. The new policy add a verification using the CRL-DP extension defined in the certificate. If this extension is defined it has precedence over the OCSP and if it pass no other check are performed. If CRL cannot be retrieved then the OCSP responder is used. This new method takes place when OCSP is configured without a default responder and the PKIX verification method is adopted (with the policy OCSP_LEAF_AND_CHAIN_POLICY). At least a verification method has to return success to accept the certificate.
1 parent 92aa62c commit b8f1b44

File tree

1 file changed

+3
-3
lines changed
  • native/src/main/native/org/mozilla/jss/ssl

1 file changed

+3
-3
lines changed

native/src/main/native/org/mozilla/jss/ssl/common.c

+3-3
Original file line numberDiff line numberDiff line change
@@ -945,22 +945,22 @@ JSSL_verifyCertPKIXInternal(CERTCertificate *cert,
945945
* we construct the policy ourselves. */
946946
PRUint64 ocsp_Enabled_Hard_Policy_LeafFlags[2] = {
947947
/* crl */
948-
CERT_REV_M_DO_NOT_TEST_USING_THIS_METHOD,
948+
CERT_REV_M_TEST_USING_THIS_METHOD,
949949
/* ocsp */
950950
CERT_REV_M_TEST_USING_THIS_METHOD |
951951
CERT_REV_M_FAIL_ON_MISSING_FRESH_INFO
952952
};
953953

954954
PRUint64 ocsp_Enabled_Hard_Policy_ChainFlags[2] = {
955955
/* crl */
956-
CERT_REV_M_DO_NOT_TEST_USING_THIS_METHOD,
956+
CERT_REV_M_TEST_USING_THIS_METHOD,
957957
/* ocsp */
958958
CERT_REV_M_TEST_USING_THIS_METHOD |
959959
CERT_REV_M_FAIL_ON_MISSING_FRESH_INFO
960960
};
961961

962962
CERTRevocationMethodIndex ocsp_Enabled_Hard_Policy_Method_Preference[1] = {
963-
cert_revocation_method_ocsp
963+
cert_revocation_method_crl
964964
};
965965

966966
CERTRevocationFlags ocsp_Enabled_Hard_Policy = {

0 commit comments

Comments
 (0)