Warning cleanup in package sun.security.ssl (2/n)

2012-11-22 Thread Florian Weimer
This patch removes unused variables. If any of them have to stay because they are accessed through reflection (possibly by legacy code), there really ought to be a comment reflecting that. The last hunk removes a NullPointerException, I think. -- Florian Weimer / Red Hat Product Security Team

TLS 1.2 signature_algorithms extension processing

2012-11-22 Thread Florian Weimer
Is the extension actually being processed? sun.security.ssl.HandshakeHash.setCertificateVerifyAlg(String) appears to have no effect because the cvAlg member is never read. -- Florian Weimer / Red Hat Product Security Team

Code Review Request for 7030966, Support AEAD CipherSuites (JSSE part of JEP 115)

2012-11-22 Thread Xuelei Fan
Hi Valerie, Max or Brad, Can you review the update for JDK-7030966? It is the JSSE part of JEP 115. webrev: http://cr.openjdk.java.net./~xuelei/7030966/webrev.00/ JEP 115: http://openjdk.java.net/jeps/115 In the update, I have not remove the debug synchronization. I will remove them before push