--- old/src/java.base/share/classes/java/security/Provider.java 2016-04-05 10:16:46.000000000 +0800 +++ new/src/java.base/share/classes/java/security/Provider.java 2016-04-05 10:16:46.000000000 +0800 @@ -142,8 +142,23 @@ Constructor con = clazz.getConstructor(); return con.newInstance(); } else { - Constructor con = clazz.getConstructor(ctrParamClz); - return con.newInstance(ctorParamObj); + // Looking for the constructor with a params first and fallback + // to one without if not found. This is to support the enhanced + // SecureRandom. Before jdk9, there was no params support + // (only getInstance(alg)) and an impl only had the no-params + // constructor. Since jdk9, there is getInstance(alg,params) + // and an impl can contain a Impl(params) constructor. + try { + Constructor con = clazz.getConstructor(ctrParamClz); + return con.newInstance(ctorParamObj); + } catch (NoSuchMethodException nsme) { + if (ctorParamObj == null) { + Constructor con = clazz.getConstructor(); + return con.newInstance(); + } else { + throw new IllegalArgumentException(nsme); + } + } } } @@ -1384,7 +1399,8 @@ addEngine("KeyPairGenerator", false, null); addEngine("KeyStore", false, null); addEngine("MessageDigest", false, null); - addEngine("SecureRandom", false, null); + addEngine("SecureRandom", false, + "java.security.SecureRandomParameters"); addEngine("Signature", true, null); addEngine("CertificateFactory", false, null); addEngine("CertPathBuilder", false, null); @@ -1678,6 +1694,7 @@ } } } + // constructorParameter can be null if not provided return newInstanceUtil(getImplClass(), ctrParamClz, constructorParameter); } catch (NoSuchAlgorithmException e) { throw e;