RFR: 8244336: Restrict algorithms at JCE layer [v9]
Artur Barashev
abarashev at openjdk.org
Mon Aug 18 18:44:18 UTC 2025
On Sat, 16 Aug 2025 00:46:13 GMT, Valerie Peng <valeriep at openjdk.org> wrote:
>> This enhancement introduces a new security property "jdk.crypto.disabledAlgorithms" which can be leveraged to disable algorithms for JCE/JCA crypto services. For now, only Cipher, KeyStore, MessageDigest, and Signature services support this new security property. The support can be expanded later to cover more services if needed. Note that this security property is meant to disable algorithms irrespective of providers. If the algorithm is found to be disabled, it will be rejected before reaching out to provider(s) for the corresponding implementation(s).
>>
>> A few implementation notes:
>> 1) The specified security property value is lazily loaded and all changes after it's been loaded are ignored. Invalid entries, e.g. wrong syntax, are ignored and removed. The algorithm name check is case-insensitive. If a disabled algorithm is known to has an object identifier (oid) by JDK, this oid and its aliases is also added to the disabled services.
>> 2) The algorithm name checking impl is based on the sun.security.util.AlgorithmConstraints class, but without the decomposing and different constraints.
>> 3) The hardwiring of NONEwithRSA signature to RSA/ECB/PKCS1Padding cipher in java.security.Signature class is removed. Instead, this is moved to the provider level, i.e. SunJCE and SunPKCS11 provider are changed to claim the NONEwithRSA signature support. Disabling one will not affect the other.
>>
>> CSR will be filed once the review is wrapping up.
>>
>> Thanks~
>> Valerie
>
> Valerie Peng has updated the pull request incrementally with one additional commit since the last revision:
>
> Address review comments from Tony and Artur.
src/java.base/share/classes/sun/security/util/CryptoAlgorithmConstraints.java line 77:
> 75: * Initialize algorithm constraints with the specified security property
> 76: * {@code propertyName}. Note that if a system property of the same name
> 77: * is set, it overrides the security property.
We allow a system property to override `jdk.crypto.disabledAlgorithms` security property but not other `*.disabledAlgorithms` security properties. That's an inconsistent experience. Any particular reason we need this functionality for `jdk.crypto.disabledAlgorithms`? Are we going to document it in JSSE guide?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26377#discussion_r2283164102
More information about the security-dev
mailing list