Re: [9] RFR 6877937: The SunJCE PBKDF2KeyImpl is requiring the MAC instance also be from SunJCE.

[Xuelei Fan]

Looks fine to me.

Xuelei

On 8/11/2016 7:55 PM, Vincent Ryan wrote:
> Please review this change to unpin the Mac implementation from the
> SunJCE provider.
> Since the Mac is a private field there are no issues
> regarding Clonable implementations for Mac or its MessageDigest.
> Thanks.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-6977937
> 
> *diff --git
> a/src/java.base/share/classes/com/sun/crypto/provider/PBKDF2KeyImpl.java
> b/src/java.base/share/classes/com/sun/crypto/provider/PBKDF2KeyImpl.java*
> *---
> a/src/java.base/share/classes/com/sun/crypto/provider/PBKDF2KeyImpl.java*
> *+++
> b/src/java.base/share/classes/com/sun/crypto/provider/PBKDF2KeyImpl.java*
> @@ -107,7 +107,7 @@
>              throw new InvalidKeySpecException("Key length is negative");
>          }
>          try {
> -            this.prf = Mac.getInstance(prfAlgo, SunJCE.getInstance());
> +            this.prf = Mac.getInstance(prfAlgo);
>          } catch (NoSuchAlgorithmException nsae) {
>              // not gonna happen; re-throw just in case
>              InvalidKeySpecException ike = new InvalidKeySpecException();