8214098: sun.security.ssl.HandshakeHash.T12HandshakeHash constructor check backwards.

Xue-Lei Fan xuelei.fan at oracle.com
Tue Dec 4 23:09:57 UTC 2018


Looks fine to me.

Thanks,
Xuelei

On 12/4/2018 12:06 PM, Anthony Scarpino wrote:
> Hi,
> 
> I need a code review for the TLS 1.2 check being backwards for 
> HandshakeHash, which showed up with a clonable MessageDigest object. 
> Since TLS 1.3 does not use the same scheme as 1.2, I removed the 
> unnecessary code.
> 
> http://cr.openjdk.java.net/~ascarpino/8214098/webrev/
> 
> Tony
> 
> 
> 



More information about the security-dev mailing list