这段代码的问题出在哪呢，出在后一行，或者说出在前面实现给usedPooled++上也可以。
　　在这里JBoss Remoting过于相信createClientSocket这行代码了，jboss remoting认为这行代码是不可能抛出异常的，但事实上其实这行是有可能会抛出异常的，可以想下，如果这行代码执行抛出异常的话，会造成的现象是之前说的，客户端连接池中的连接被占用了一个，而且没有回收的地方。
　　所以简单的方法自然是在pooled=createClientSocket(socket， address.timeout， metadata);这行代码上增加捕捉try...catch，如果有异常抛出的话，则将usedPooled--；像之前createSocket那个地方一样。
　　在JBoss Remoting 2.4中，jboss不再采用usedPooled这个long型加上usedPoolLock这个对象锁的方式来控制连接池，而是改为了采用更简单好用的Semphore，不过用的还是EDG包的，而不是java 5的，来看看jboss remoting 2.4中的这段代码改成什么样了：
boolean timedout = !semaphore.attempt(timeToWait);
if (trace) log.trace(this + " obtained semaphore: " + semaphore.permits());
if (timedout)
{
throw new IllegalStateException("Timeout waiting for a free socket");
}
SocketWrapper pooled = null;
if (tryPool)
{
synchronized (pool)
{
// if connection within pool， use it
if (pool.size() > 0)
{
pooled = getPooledConnection();
if (trace) log.trace(this + " reusing pooled connection: " + pooled);
}
}
}
else
{
if (trace) log.trace(this + " avoiding connection pool， creating new socket");
}
if (pooled == null)
{
//Need to create a new one
Socket socket = null;
if (trace) { log.trace(this + " creating socket "); }
// timeAllowed < 0 indicates no per invocation timeout has been set.
int timeRemaining = -1;
if (0 <= timeAllowed)
{
timeRemaining = (int) (timeAllowed - (System.currentTimeMillis() - start));
}
socket = createSocket(address.address， address.port， timeRemaining);
if (trace) log.trace(this + " created socket: " + socket);
socket.setTcpNoDelay(address.enableTcpNoDelay);
Map metadata = getLocator().getParameters();
if (metadata == null)
{
metadata = new HashMap(2);
}
else
{
metadata = new HashMap(metadata);
}
metadata.put(SocketWrapper.MARSHALLER， marshaller);
metadata.put(SocketWrapper.UNMARSHALLER， unmarshaller);
if (timeAllowed > 0)
{
timeRemaining = (int) (timeAllowed - (System.currentTimeMillis() - start));
if (timeRemaining <= 0)
throw new IllegalStateException("Timeout creating a new socket");
metadata.put(SocketWrapper.TEMP_TIMEOUT， new Integer(timeRemaining));
}
pooled = createClientSocket(socket， address.timeout， metadata);
}
return pooled;