On Wed, Apr 23, 2014 at 11:29 PM, Jonathon Jongsma <jjong...@redhat.com>wrote:

> unrelated format changes...
>
> ok, moved

>
> >  {
> >      SpiceUsbDeviceManager *self;
> >      SpiceUsbDeviceManagerPrivate *priv;
> > @@ -260,6 +260,17 @@ static gboolean
> > spice_usb_device_manager_initable_init(GInitable  *initable,
> >      }
> >
> >  #ifdef USE_USBREDIR
> > +
> > +#ifdef G_OS_WIN32
> > +    priv->installer = spice_win_usb_driver_new();
> > +    if (!priv->installer) {
>
>
> Am I missing something?  How can this fail?  spice_win_usb_driver_new() is
> just a simple wrapper around g_object_new()...
>

The following patches will call g_initalble_new() which can return NULL.


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

Reply via email to