Mercurial > hg > icedtea8-forest > jdk

changeset 13671:5eaf8c860c22 jdk8u191-b11

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
8211107: LDAPS communication failure with jdk 1.8.0_181 Reviewed-by: chegar, coffeys, vtewari
author pkoppula
date Tue, 02 Oct 2018 21:38:54 +0530
parents 8378d1047fa2
children 5f60b2d7a15d 2cd82eb879dd
files src/share/classes/com/sun/jndi/ldap/Connection.java
diffstat 1 files changed, 3 insertions(+), 3 deletions(-) [+]
line wrap: on
line diff
--- a/src/share/classes/com/sun/jndi/ldap/Connection.java	Thu Sep 27 05:13:17 2018 -0700
+++ b/src/share/classes/com/sun/jndi/ldap/Connection.java	Tue Oct 02 21:38:54 2018 +0530
@@ -382,17 +382,17 @@
         // then reset the timeout.
         if (socket instanceof SSLSocket) {
             SSLSocket sslSocket = (SSLSocket) socket;
-            int socketTimeout = sslSocket.getSoTimeout();
             if (!IS_HOSTNAME_VERIFICATION_DISABLED) {
                 SSLParameters param = sslSocket.getSSLParameters();
                 param.setEndpointIdentificationAlgorithm("LDAPS");
                 sslSocket.setSSLParameters(param);
             }
             if (connectTimeout > 0) {
+                int socketTimeout = sslSocket.getSoTimeout();
                 sslSocket.setSoTimeout(connectTimeout); // reuse full timeout value
+                sslSocket.startHandshake();
+                sslSocket.setSoTimeout(socketTimeout);
             }
-            sslSocket.startHandshake();
-            sslSocket.setSoTimeout(socketTimeout);
         }
         return socket;
     }