On Tue, Nov 15, 2016 at 02:33:15PM +0100, Victor Toso wrote: > From: Victor Toso <me@xxxxxxxxxxxxxx> > > 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() Much improved log, thanks! Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx> > > Resolves: https://bugs.freedesktop.org/show_bug.cgi?id=98686 > Signed-off-by: Victor Toso <victortoso@xxxxxxxxxx> > Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx> > --- > src/win-usb-dev.c | 3 --- > 1 file changed, 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; > } > > -- > 2.9.3 > > _______________________________________________ > Spice-devel mailing list > Spice-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/spice-devel
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel