GUACAMOLE-958: Avoid race/deadlock in guacd_timed_client_free()
diff --git a/src/guacd/proc.c b/src/guacd/proc.c
index 27ad69c..aacf2b0 100644
--- a/src/guacd/proc.c
+++ b/src/guacd/proc.c
@@ -267,24 +267,24 @@
.tv_nsec = current_time.tv_usec * 1000
};
- /* Free the client in a separate thread, so we can time the free operation */
- if (pthread_create(&client_free_thread, NULL,
- guacd_client_free_thread, &free_operation))
- return 1;
-
/* The mutex associated with the pthread conditional and flag MUST be
* acquired before attempting to wait for the condition */
if (pthread_mutex_lock(&free_operation.completed_mutex))
return 1;
- /* Wait a finite amount of time for the free operation to finish */
- if (pthread_cond_timedwait(&free_operation.completed_cond,
- &free_operation.completed_mutex, &deadline))
- return 1;
+ /* Free the client in a separate thread, so we can time the free operation */
+ if (!pthread_create(&client_free_thread, NULL,
+ guacd_client_free_thread, &free_operation)) {
+
+ /* Wait a finite amount of time for the free operation to finish */
+ (void) pthread_cond_timedwait(&free_operation.completed_cond,
+ &free_operation.completed_mutex, &deadline);
+ }
+
+ (void) pthread_mutex_unlock(&free_operation.completed_mutex);
/* Return status of free operation */
return !free_operation.completed;
-
}
/**