andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

[#2589] LocalServerChannel.doClose() throws NPE when localAddress == null

Browse Source 
Motivation:

LocalServerChannel.doClose() calls LocalChannelRegistry.unregister(localAddress); without check if localAddress is null and so produce a NPE when pass null the used ConcurrentHashMapV8

Modification:
Check for localAddress != null before try to remove it from Map. Also added a unit test which showed the stacktrace of the error.

Result:

No more NPE during doClose().
This commit is contained in:
Norman Maurer 2014-06-20 20:03:22 +02:00
parent c44b5f54fb
commit fa5e602238
2 changed files with 30 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
transport/src/main/java/io/netty/channel/local/LocalServerChannel.java
View File

@ -96,8 +96,10 @@ public class LocalServerChannel extends AbstractServerChannel {
protected void doClose() throws Exception {
if (state <= 1) {
// Update all internal state before the closeFuture is notified.
LocalChannelRegistry.unregister(localAddress);
localAddress = null;
if (localAddress != null) {
LocalChannelRegistry.unregister(localAddress);
localAddress = null;
}
state = 2;
}
}

26
transport/src/test/java/io/netty/bootstrap/BootstrapTest.java
View File

@ -161,6 +161,32 @@ public class BootstrapTest {
}
}
@Test
public void testLateRegisterFailed() throws Exception {
final TestEventLoopGroup group = new TestEventLoopGroup();
try {
ServerBootstrap bootstrap = new ServerBootstrap();
bootstrap.group(group);
bootstrap.channel(LocalServerChannel.class);
bootstrap.childHandler(new DummyHandler());
bootstrap.localAddress(new LocalAddress("1"));
ChannelFuture future = bootstrap.bind();
Assert.assertFalse(future.isDone());
group.promise.setFailure(new IllegalStateException());
final BlockingQueue<Boolean> queue = new LinkedBlockingQueue<Boolean>();
future.addListener(new ChannelFutureListener() {
@Override
public void operationComplete(ChannelFuture future) throws Exception {
queue.add(group.next().inEventLoop(Thread.currentThread()));
}
});
Assert.assertFalse(queue.take());
} finally {
group.shutdownGracefully();
group.terminationFuture().sync();
}
}
private static final class TestEventLoopGroup extends DefaultEventLoopGroup {
ChannelPromise promise;
TestEventLoopGroup() {