On Thu, 2 Sep 2021 13:33:30 GMT, Fernando Guallini <fguall...@openjdk.org> 
wrote:

> This trivial change fixes the case 2 in test 
> sun/security/pkcs11/rsa/TestP11KeyFactoryGetRSAKeySpec, the method 
> testKeySpec is expecting a class object of type KeySpec as second argument in 
> order to be reusable for multiple test scenarios, but then instead of using 
> that argument the RSAPrivateKeySpec.class is hardcoded:
> 
> 
> private static void testKeySpec(KeyFactory factory, PrivateKey key, Class<? 
> extends KeySpec> specClass) throws Exception {
>         try {
>             KeySpec spec = factory.getKeySpec(key, RSAPrivateKeySpec.class);
> 
> 
> it should be:
> 
> `KeySpec spec = factory.getKeySpec(key, specClass);
> `

Marked as reviewed by mullan (Reviewer).

-------------

PR: https://git.openjdk.java.net/jdk/pull/5351

Reply via email to