Skip to content

Commit a71fee4

Browse files
author
Kelly Byrd
committed
Fix NPE in NettyAsyncHttpProvider.toString()
When maxConnectionsTotal isn't set, freeConnections is always null.
1 parent a35881e commit a71fee4

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

src/main/java/com/ning/http/client/providers/netty/NettyAsyncHttpProvider.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -271,8 +271,9 @@ public NettyAsyncHttpProvider(AsyncHttpClientConfig config) {
271271

272272
@Override
273273
public String toString() {
274+
int availablePermits = freeConnections != null ? freeConnections.availablePermits() : 0;
274275
return String.format("NettyAsyncHttpProvider:\n\t- maxConnections: %d\n\t- openChannels: %s\n\t- connectionPools: %s",//
275-
config.getMaxTotalConnections() - freeConnections.availablePermits(),//
276+
config.getMaxTotalConnections() - availablePermits,//
276277
openChannels.toString(),//
277278
connectionsPool.toString());
278279
}

0 commit comments

Comments
 (0)