Blame SOURCES/0029-channel-main-Copy-SpiceMigrationDstInfo-into-spice_m.patch

76100f
From ab42be2b00d12d0bc98c6ddea08a7f969e83b2ac Mon Sep 17 00:00:00 2001
76100f
From: Frediano Ziglio <freddy77@gmail.com>
76100f
Date: Mon, 10 Aug 2020 15:35:26 +0100
76100f
Subject: [PATCH 29/31] channel-main: Copy SpiceMigrationDstInfo into
76100f
 spice_migrate
76100f
76100f
The message could disappear while the structure is used.
76100f
76100f
Signed-off-by: Frediano Ziglio <freddy77@gmail.com>
76100f
Acked-by: Uri Lublin <uril@redhat.com>
76100f
---
76100f
 src/channel-main.c | 15 +++++++++++----
76100f
 1 file changed, 11 insertions(+), 4 deletions(-)
76100f
76100f
diff --git a/src/channel-main.c b/src/channel-main.c
76100f
index 8caf727..5f81975 100644
76100f
--- a/src/channel-main.c
76100f
+++ b/src/channel-main.c
76100f
@@ -121,7 +121,7 @@ struct _SpiceMainChannelPrivate  {
76100f
 
76100f
 struct spice_migrate {
76100f
     struct coroutine *from;
76100f
-    SpiceMigrationDstInfo *info;
76100f
+    SpiceMigrationDstInfo info;
76100f
     SpiceSession *session;
76100f
     int ref_count;
76100f
     guint nchannels;
76100f
@@ -2258,6 +2258,8 @@ static void
76100f
 spice_migrate_unref(spice_migrate *mig)
76100f
 {
76100f
     if (mig != NULL && --mig->ref_count == 0) {
76100f
+        g_free(mig->info.host_data);
76100f
+        g_free(mig->info.cert_subject_data);
76100f
         g_free(mig);
76100f
     }
76100f
 }
76100f
@@ -2396,7 +2398,6 @@ static gboolean migrate_connect(spice_migrate *mig)
76100f
     const char *host;
76100f
 
76100f
     g_return_val_if_fail(mig != NULL, FALSE);
76100f
-    g_return_val_if_fail(mig->info != NULL, FALSE);
76100f
     g_return_val_if_fail(mig->nchannels == 0, FALSE);
76100f
     c = SPICE_CHANNEL(mig->src_channel)->priv;
76100f
     g_return_val_if_fail(c != NULL, FALSE);
76100f
@@ -2404,7 +2405,7 @@ static gboolean migrate_connect(spice_migrate *mig)
76100f
 
76100f
     spice_session_set_migration_state(mig->session, SPICE_SESSION_MIGRATION_CONNECTING);
76100f
 
76100f
-    SpiceMigrationDstInfo *info = mig->info;
76100f
+    SpiceMigrationDstInfo *info = &mig->info;
76100f
     SPICE_DEBUG("migrate_begin %u %s %d %d",
76100f
                 info->host_size, info->host_data, info->port, info->sport);
76100f
     port = info->port;
76100f
@@ -2461,7 +2462,13 @@ static void main_migrate_connect(SpiceChannel *channel,
76100f
     mig = spice_new0(spice_migrate, 1);
76100f
     mig->ref_count = 1;
76100f
     mig->src_channel = channel;
76100f
-    mig->info = dst_info;
76100f
+    mig->info = *dst_info;
76100f
+    if (dst_info->host_data) {
76100f
+        mig->info.host_data = (void *) g_strdup((char*) dst_info->host_data);
76100f
+    }
76100f
+    if (dst_info->cert_subject_data) {
76100f
+        mig->info.cert_subject_data = (void *) g_strdup((char*) dst_info->cert_subject_data);
76100f
+    }
76100f
     mig->from = coroutine_self();
76100f
     mig->do_seamless = do_seamless;
76100f
     mig->src_mig_version = src_mig_version;
76100f
-- 
76100f
2.28.0
76100f