Mailing-List: contact commits-help@harmony.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@harmony.apache.org
Message-ID: <23943331.435281285644770405.JavaMail.jira@thor>
Date: Mon, 27 Sep 2010 23:32:50 -0400 (EDT)
From: "Wendy Feng (JIRA)" <jira@apache.org>
To: commits@harmony.apache.org
Subject: [jira] Created: (HARMONY-6662) Spin wait in
 StartTlsResponseImpl.java
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

Spin wait in StartTlsResponseImpl.java
--------------------------------------

                 Key: HARMONY-6662
                 URL: https://issues.apache.org/jira/browse/HARMONY-6662
             Project: Harmony
          Issue Type: Bug
          Components: Classlib
    Affects Versions: 6.0M1
         Environment: Windows XP
            Reporter: Wendy Feng


I found spin wait in modules/jndi/src/main/java/org/apache/harmony/jndi/provider/ldap/ext/StartTlsResponseImpl.java

public class StartTlsResponseImpl extends StartTlsResponse {
...
  public SSLSession negotiate(SSLSocketFactory factory) throws IOException {
   ...
  sslSocket.startHandshake();

        while (!isHandshaked) {
            // Wait for handshake finish.
        }
   ...
   }
...
}
  
Spin wait for isHandshaked is updated by other threads.

Consequence:
Bad performance due to exhaustive use of CPU time.
In current Java memory model, even if the isHandshaked field is updated by other threads, current thread that spins checking for isHandshaked may not see the up-to-date value, turn the code into an infinite loop.

I suggest to use wait/notify for synchronization as follow
public class StartTlsResponseImpl extends StartTlsResponse {
private static final Object monitor = new Object():
...
  public SSLSession negotiate(SSLSocketFactory factory) throws IOException {
   ...
  sslSocket.startHandshake();

        synchronized(monitor){
              while (!isHandshaked) {
                  monitor.wait();
              }
        }
   ...
   }
...
}

Pls make sure to change the code that will update the value of isHandshaked correspondingly.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.