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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxx>
>
> 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@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

[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]