channel is a SocketChannel which is a subclass of SelectableChannel, so this instanceof
can't ever be false. (And just above, we've already confirmed that channel != null, so this check wasn't falling back to protecting us from that) git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@4867 542714f4-19e9-0310-aa3c-eee0fc999fb1
This commit is contained in:
@@ -656,18 +656,6 @@ public class ConnectionManager extends LoopingThread
|
|||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!(channel instanceof SelectableChannel)) {
|
|
||||||
try {
|
|
||||||
log.warning("Provided with un-selectable socket as result of accept(), can't " +
|
|
||||||
"cope [channel=" + channel + "].");
|
|
||||||
} catch (Error err) {
|
|
||||||
log.warning("Un-selectable channel also couldn't be printed.");
|
|
||||||
}
|
|
||||||
// stick a fork in the socket
|
|
||||||
channel.socket().close();
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
// log.fine("Accepted connection " + channel + ".");
|
// log.fine("Accepted connection " + channel + ".");
|
||||||
|
|
||||||
// create a new authing connection object to manage the authentication of this client
|
// create a new authing connection object to manage the authentication of this client
|
||||||
|
|||||||
Reference in New Issue
Block a user