close callbacks prior to closing connection to avoid potential race when e.g timer callback and connection_destroy run in parallel

This commit is contained in:
Vladislav Vaintroub 2012-01-27 00:39:23 +01:00
commit 8b945a1419

View file

@ -404,11 +404,6 @@ void set_wait_timeout(connection_t *connection, ulonglong old_timeout)
/* Connection destructor */
void destroy_connection(connection_t *connection)
{
if (connection->thd)
{
threadpool_remove_connection(connection->thd);
}
if (connection->io)
{
WaitForThreadpoolIoCallbacks(connection->io, TRUE);
@ -427,6 +422,11 @@ void destroy_connection(connection_t *connection)
WaitForThreadpoolTimerCallbacks(connection->timer, TRUE);
CloseThreadpoolTimer(connection->timer);
}
if (connection->thd)
{
threadpool_remove_connection(connection->thd);
}
DestroyThreadpoolEnvironment(&connection->callback_environ);
}