Re: [9] RFR 8157881: security.provider property description needs to be updated for modules

2016-06-13 Thread Sean Mullan
On 06/10/2016 04:07 PM, Valerie Peng wrote: Sounds good. Webrev updated at: http://cr.openjdk.java.net/~valeriep/8157881/webrev.01/ Looks good. --Sean

[9] RFR: 8159038: javax/net/ssl/SSLSession/SessionCacheSizeTests.java failed with java.net.SocketException: Address already in use

2016-06-13 Thread Artem Smotrakov
Hello, Please review this patch for 9. javax/net/ssl/SSLSession/SessionCacheSizeTests.java test fails intermittently with "java.net.SocketException: Address already in use" exception. This exception occurs while creating a server socket: ... void doServerSide(int serverPort, int serverConns)

[9] RFR 8154191: Deprivilege java.smartcardio module

2016-06-13 Thread Valerie Peng
Sean, Can you please review the changes for deprivileging the java.smartcardio module? I have to update one makefile in the top-level workspace besides the java.policy file in the jdk workspace. One java.smartcardio regression test uses custom policy file which needs to be updated as well. B

Re: RFR 8130302: jarsigner and keytool -providerClass needs be re-examined for modules

2016-06-13 Thread Wang Weijun
OK, please take a review at the new version at http://cr.openjdk.java.net/~weijun/8130302/webrev.04/ Changes from webrev.03: 1. The new option name -addprovider is used, along with the changes in Resources.java. 2. In KeyStoreUtil::loadProviderByClass, special treatment for "sun.security.pk