Mailing-List: contact dev-help@hc.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "HttpComponents Project" <dev@hc.apache.org>
Date: Fri, 31 Aug 2012 07:35:07 +1100 (NCT)
From: "Jonathan Burke (JIRA)" <jira@apache.org>
To: dev@hc.apache.org
Message-ID: <2123545851.18139.1346358907841.JavaMail.jiratomcat@arcas>
Subject: [jira] [Created] (HTTPCLIENT-1229) @ThreadSafe
 BasicClientConnectionManager is not threadsafe with respect to shutdown
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

Jonathan Burke created HTTPCLIENT-1229:
------------------------------------------

             Summary: @ThreadSafe BasicClientConnectionManager is not threadsafe with respect to shutdown
                 Key: HTTPCLIENT-1229
                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-1229
             Project: HttpComponents HttpClient
          Issue Type: Bug
          Components: HttpConn
    Affects Versions: 4.2.1
            Reporter: Jonathan Burke
            Priority: Minor


Since the assignment to variable shutdown in the shutdown method (line 262) is not part of the synchronize block and none of the assertNotShutdown method calls are within the synchronized blocks of their enclosing methods, it is possible to have threads execute the commands of BasicClientConnectionManager methods in the following sequence.

Thread 1
releaseConnection (or some other method that uses assertNotShutdown is called)
  assertNotShutdown - Passes - Line 183

Thread 2
shutdown
  the shutdown flag becomes true - Line 262
  shutdown releases this.poolEntry and this.conn in the synchronized(this) block
  
Thread 1
  release connection's synchronized(this) block get's executed 
  	this.poolEntry is null in the try block causing a null pointer exception - Line 211
  	this.poolEntry is null in the finally block causing a null pointer exception - Line 224

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org