cmake-server: Stop the file monitor on client disconnect

When the client disconnects we need to remove all events from our loop.
Prior to the introduction of the file monitor we only needed to remove
the client pipes.  Now we need to remove the file monitor events too.

Without this the event loop may continue to block on file monitor
events.  If one does eventually come in then the event handler may try
to report it to the disconnected client and crash because our internal
structures for writing to the client have been freed.

This addresses a failure of the `Server` test on some machines.
This commit is contained in:
Brad King 2016-09-30 15:01:29 -04:00
parent 8491a539cf
commit 1b43f9d12d
1 changed files with 1 additions and 0 deletions

View File

@ -164,6 +164,7 @@ void cmServerConnection::ReadData(const std::string& data)
void cmServerConnection::HandleEof()
{
this->FileMonitor()->StopMonitoring();
this->TearDown();
}