|
 |
3538ab |
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
|
|
|
3538ab |
From: Victor Toso <me@victortoso.com>
|
|
|
3538ab |
Date: Mon, 11 Jun 2018 12:14:30 +0200
|
|
|
3538ab |
Subject: [PATCH] Revert "channel-usbredir: Fix crash on channel-up"
|
|
|
3538ab |
|
|
|
3538ab |
This reverts commit 291f3e4419e6fb4077ae43a5e09eb1c37b9dd729.
|
|
|
3538ab |
|
|
|
3538ab |
Follow up patch should address rhbz#1399838 mentioned in commit above
|
|
|
3538ab |
in a different way.
|
|
|
3538ab |
|
|
|
3538ab |
Major reason to revert is that the SpiceUsbDeviceManager object is
|
|
|
3538ab |
kept in SpiceSession as an easy way to share it between different
|
|
|
3538ab |
SpiceUsbredirChannel while SpiceSession itself does not use it. This
|
|
|
3538ab |
causes problems on migration as we start a new session with a new
|
|
|
3538ab |
SpiceUsbDeviceManager object while the previous one still exists.
|
|
|
3538ab |
|
|
|
3538ab |
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1558043
|
|
|
3538ab |
Signed-off-by: Victor Toso <victortoso@redhat.com>
|
|
|
3538ab |
Acked-by: Christophe Fergeau <cfergeau@redhat.com>
|
|
|
3538ab |
---
|
|
|
3538ab |
src/spice-session.c | 7 -------
|
|
|
3538ab |
1 file changed, 7 deletions(-)
|
|
|
3538ab |
|
|
|
3538ab |
diff --git a/src/spice-session.c b/src/spice-session.c
|
|
|
3538ab |
index a729cc3..094dc41 100644
|
|
|
3538ab |
--- a/src/spice-session.c
|
|
|
3538ab |
+++ b/src/spice-session.c
|
|
|
3538ab |
@@ -281,7 +281,6 @@ static void spice_session_init(SpiceSession *session)
|
|
|
3538ab |
{
|
|
|
3538ab |
SpiceSessionPrivate *s;
|
|
|
3538ab |
gchar *channels;
|
|
|
3538ab |
- GError *err = NULL;
|
|
|
3538ab |
|
|
|
3538ab |
SPICE_DEBUG("New session (compiled from package " PACKAGE_STRING ")");
|
|
|
3538ab |
s = session->priv = SPICE_SESSION_GET_PRIVATE(session);
|
|
|
3538ab |
@@ -294,12 +293,6 @@ static void spice_session_init(SpiceSession *session)
|
|
|
3538ab |
s->images = cache_image_new((GDestroyNotify)pixman_image_unref);
|
|
|
3538ab |
s->glz_window = glz_decoder_window_new();
|
|
|
3538ab |
update_proxy(session, NULL);
|
|
|
3538ab |
-
|
|
|
3538ab |
- s->usb_manager = spice_usb_device_manager_get(session, &err;;
|
|
|
3538ab |
- if (err != NULL) {
|
|
|
3538ab |
- SPICE_DEBUG("Could not initialize SpiceUsbDeviceManager - %s", err->message);
|
|
|
3538ab |
- g_clear_error(&err;;
|
|
|
3538ab |
- }
|
|
|
3538ab |
}
|
|
|
3538ab |
|
|
|
3538ab |
static void
|