[cp-patches] FYI: HTTPConnection.Pool fixlet

David Daney ddaney at avtrex.com
Sat Mar 4 23:04:50 UTC 2006


Mark Wielaard wrote:
> Hi,
> 
> If we reused a connection from the HTTPConnection.Pool we forgot to
> actually remove that connection from the pool itself. That could lead to
> the connection being closed by the Reaper when it was used, or if it was
> returned to the pool it would be inserted in the LinkedList twice which
> meant we could be returning the same connection (or an already closed
> connection). Fixed as follows:
> 
> 2006-03-04  Mark Wielaard  <mark at klomp.org>
> 
>     * gnu/java/net/protocol/http/HTTPConnection.java (Pool.get): Remove
>     existing connection from pool before returning.
> 
> This should solve the issue Christian was seeing.
> 

Sorry about that :(  Your fix is obviously correct.

David Daney.



More information about the Classpath-patches mailing list