On 07/09/2012 07:47 PM, Marc-André Lureau wrote:
Hi

On Mon, Jul 9, 2012 at 2:14 PM, Uri Lublin<u...@redhat.com>  wrote:
+void spice_usb_device_manager_connect_device_async(SpiceUsbDeviceManager *self,
+                                             SpiceUsbDevice *device,
+                                             GCancellable *cancellable,
+                                             GAsyncReadyCallback callback,
+                                             gpointer user_data)
+{
+
+#if defined(USE_USBREDIR)&&  defined(G_OS_WIN32)
+    SpiceWinUsbDriver *installer;
+    UsbInstallCbInfo *cbinfo;
+
+    installer = spice_win_usb_driver_new();
+    cbinfo = g_new0(UsbInstallCbInfo, 1);
Who is freeing installer? In fact, I think we could simplify the
cbinfo + installer and use just a single object. But we may want to
leave that for a later clean up.

spice_usb_device_manager_drv_install_cb()  frees the installer.

_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to