[Spice-devel] [spice-gtk v1 1/2] Revert "channel-usbredir: Fix crash on channel-up"

Victor Toso victortoso at redhat.com
Mon Jun 11 10:04:30 UTC 2018


Hi,

On Mon, Jun 11, 2018 at 11:50:05AM +0200, Christophe Fergeau wrote:
> Fine with me, hopefully we won't forget to fix this for good :-/

https://gitlab.freedesktop.org/spice/spice-gtk/issues/67

> Acked-by: Christophe Fergeau <cfergeau at redhat.com>

Thanks, I'll be pushing this shortly!

Cheers,

> Christophe
> 
> On Mon, Jun 11, 2018 at 09:11:40AM +0200, Victor Toso wrote:
> > Hi,
> > 
> > Could we have this revert for the release and then I'll open a
> > bug to fix it properly during the next release?
> > 
> > Once more, the patch that is being reverted fixes a crash on
> > channel-up of channel-usbredir.c while running spicy-stats tool,
> > see [0] - The crash can be avoided with a check done in the 2nd
> > patch.
> > 
> > [0] https://bugzilla.redhat.com/show_bug.cgi?id=1399838
> > 
> > I tried to fix this without reverting but failed for the time
> > allocated at it.
> > 
> > Cheers,
> >         toso
> > 
> > On Fri, Apr 06, 2018 at 09:59:43AM +0200, Victor Toso wrote:
> > > From: Victor Toso <me at victortoso.com>
> > >
> > > This reverts commit 291f3e4419e6fb4077ae43a5e09eb1c37b9dd729
> > >
> > > Follow up patch should address rhbz#1399838 mentioned in commit above
> > > in a different way.
> > >
> > > Major reason to revert is that the SpiceUsbDeviceManager object is
> > > kept in SpiceSession as an easy way to share it between different
> > > SpiceUsbredirChannel while SpiceSession itself does not use it. This
> > > causes problems on migration as we start a new session with a new
> > > SpiceUsbDeviceManager object while the previous one still exists.
> > > 
> > > Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1558043
> > > ---
> > >  src/spice-session.c | 7 -------
> > >  1 file changed, 7 deletions(-)
> > > 
> > > diff --git a/src/spice-session.c b/src/spice-session.c
> > > index 57acc63..f856244 100644
> > > --- a/src/spice-session.c
> > > +++ b/src/spice-session.c
> > > @@ -280,7 +280,6 @@ static void spice_session_init(SpiceSession *session)
> > >  {
> > >      SpiceSessionPrivate *s;
> > >      gchar *channels;
> > > -    GError *err = NULL;
> > >  
> > >      SPICE_DEBUG("New session (compiled from package " PACKAGE_STRING ")");
> > >      s = session->priv = SPICE_SESSION_GET_PRIVATE(session);
> > > @@ -293,12 +292,6 @@ static void spice_session_init(SpiceSession *session)
> > >      s->images = cache_image_new((GDestroyNotify)pixman_image_unref);
> > >      s->glz_window = glz_decoder_window_new();
> > >      update_proxy(session, NULL);
> > > -
> > > -    s->usb_manager = spice_usb_device_manager_get(session, &err);
> > > -    if (err != NULL) {
> > > -        SPICE_DEBUG("Could not initialize SpiceUsbDeviceManager - %s", err->message);
> > > -        g_clear_error(&err);
> > > -    }
> > >  }
> > >  
> > >  static void
> > > -- 
> > > 2.16.2
> > > 
> > > _______________________________________________
> > > Spice-devel mailing list
> > > Spice-devel at lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/spice-devel
> 
> 
> 
> > _______________________________________________
> > Spice-devel mailing list
> > Spice-devel at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/spice-devel
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20180611/f0907a87/attachment.sig>


More information about the Spice-devel mailing list