THRIFT-4161: TNonblockingServer: Fix using uninitialized event_
Client: cpp
When there are more than one IO threads, and we have failed to notify
one IO threads, then we have to close the connection. But the event_
in that connection isn't initialized. We should use setIdle() instead.
This closes #1216
diff --git a/lib/cpp/src/thrift/server/TNonblockingServer.cpp b/lib/cpp/src/thrift/server/TNonblockingServer.cpp
index c03327d..905c5d2 100644
--- a/lib/cpp/src/thrift/server/TNonblockingServer.cpp
+++ b/lib/cpp/src/thrift/server/TNonblockingServer.cpp
@@ -821,10 +821,7 @@
* Closes a connection
*/
void TNonblockingServer::TConnection::close() {
- if (eventFlags_ && event_del(&event_) == -1) {
- GlobalOutput.perror("TConnection::close() event_del", THRIFT_GET_SOCKET_ERROR);
- return;
- }
+ setIdle();
if (serverEventHandler_) {
serverEventHandler_->deleteContext(connectionContext_, inputProtocol_, outputProtocol_);