Qortal/altcoinj
1
0
Fork 1
mirror of https://github.com/Qortal/altcoinj.git synced 2025-11-02 21:47:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

Handle getting a null from getNewParser gracefully

Browse Source
This commit is contained in:
Matt Corallo
2014-08-17 21:20:46 +00:00
parent e2dbd2a3f7
commit d3003622f2
1 changed files with 10 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
core/src/main/java/com/google/bitcoin/net/NioServer.java
View File

@@ -23,6 +23,7 @@ import java.nio.channels.spi.SelectorProvider;
import java.util.Iterator;
import com.google.common.annotations.VisibleForTesting;
import com.google.common.base.Throwables;
import com.google.common.util.concurrent.AbstractExecutionThreadService;
import org.slf4j.LoggerFactory;
@@ -45,9 +46,15 @@ public class NioServer extends AbstractExecutionThreadService {
SocketChannel newChannel = sc.accept();
newChannel.configureBlocking(false);
SelectionKey newKey = newChannel.register(selector, SelectionKey.OP_READ);
ConnectionHandler handler = new ConnectionHandler(parserFactory, newKey);
newKey.attach(handler);
handler.parser.connectionOpened();
try {
ConnectionHandler handler = new ConnectionHandler(parserFactory, newKey);
newKey.attach(handler);
handler.parser.connectionOpened();
} catch (IOException e) {
// This can happen if ConnectionHandler's call to get a new handler returned null
log.error("Error handling new connection", Throwables.getRootCause(e).getMessage());
newKey.channel().close();
}
} else { // Got a closing channel or a channel to a client connection
ConnectionHandler.handleKey(key);
}