diff options
| author | Fabiano Fidêncio <fidencio@redhat.com> | 2015-07-22 03:53:41 +0200 |
|---|---|---|
| committer | Fabiano Fidêncio <fidencio@redhat.com> | 2015-07-22 14:56:57 +0200 |
| commit | 0e0f021dec03bc7a8ef776c258b886d616e65eb2 (patch) | |
| tree | 399d2bdaa916d9a930b440cfed39c825c76787ed /src | |
| parent | d2c136b020223abbf591915e06dd00fd906764c6 (diff) | |
events: don't hold events lock when dispatching free callbacks
The _event_timeout_remove and _event_handle_remove methods
were holding onto the global lock when invoking the free
callback. This is a violation of the libvirt events API
contract which requires free callbacks to be invoked in
a re-entrant safe context.
Based on commit dd17c3cc587c73a8c915238f9d9a3e200e89c93f from
libvirt-glib.
Original author: Daniel P. Berrange <berrange@redhat.com>
Related to: rhbz#1243228
Diffstat (limited to 'src')
| -rw-r--r-- | src/virt-viewer-events.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/virt-viewer-events.c b/src/virt-viewer-events.c index 1655a70..f68791f 100644 --- a/src/virt-viewer-events.c +++ b/src/virt-viewer-events.c @@ -203,13 +203,11 @@ virt_viewer_events_cleanup_handle(gpointer user_data) g_debug("Cleanup of handle %p", data); g_return_val_if_fail(data != NULL, FALSE); - g_mutex_lock(eventlock); - if (data->ff) (data->ff)(data->opaque); + g_mutex_lock(eventlock); g_ptr_array_remove_fast(handles, data); - g_mutex_unlock(eventlock); return FALSE; @@ -382,13 +380,11 @@ virt_viewer_events_cleanup_timeout(gpointer user_data) g_debug("Cleanup of timeout %p", data); g_return_val_if_fail(data != NULL, FALSE); - g_mutex_lock(eventlock); - if (data->ff) (data->ff)(data->opaque); + g_mutex_lock(eventlock); g_ptr_array_remove_fast(timeouts, data); - g_mutex_unlock(eventlock); return FALSE; |