diff options
author | Victor Toso <me@victortoso.com> | 2016-11-14 23:50:03 +0100 |
---|---|---|
committer | Victor Toso <me@victortoso.com> | 2016-11-15 17:37:36 +0100 |
commit | bd530d0feeedff39fbf8cafedd011a7abc53ac01 (patch) | |
tree | 94edf0ba08f1e18c88d143af17d380523a201b58 | |
parent | d6275ccfc43993365b601aaab6238e7a447c90d8 (diff) |
win-usb-dev: let _finalize() do libusb_exit()
This fixes a hang in mingw-spice-gtk when the GUdevClient fails to
initialize due to problems in the UsbDk installation.
Before this patch, libusb_exit() was being called twice:
- At g_udev_client_initable_init(), after failure with priv->ctx as
argument
- At g_udev_client_finalize() as g_udev_client_initable_init() failure
implies the object failed to initialize and it is terminated. Here
it was being called with NULL as priv->ctx was set to NULL in
g_udev_client_initable_init()
Resolves: https://bugs.freedesktop.org/show_bug.cgi?id=98686
Signed-off-by: Victor Toso <victortoso@redhat.com>
Acked-by: Christophe Fergeau <cfergeau@redhat.com>
-rw-r--r-- | src/win-usb-dev.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/win-usb-dev.c b/src/win-usb-dev.c index 4b95b2d..5e5bb4d 100644 --- a/src/win-usb-dev.c +++ b/src/win-usb-dev.c @@ -232,9 +232,6 @@ g_udev_client_initable_init(GInitable *initable, GCancellable *cancellable, g_udev_client_init_failed_unreg: UnregisterClass(G_UDEV_CLIENT_WINCLASS_NAME, NULL); g_udev_client_init_failed: - libusb_exit(priv->ctx); - priv->ctx = NULL; - return FALSE; } |