From 76c59470ef3ad6b76ce5d8f07e5e02df7396b92d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marc-Andr=C3=A9=20Lureau?= <marcandre.lureau@redhat.com>
Date: Wed, 27 Jan 2021 14:19:56 +0400
Subject: [PATCH] worker: ignore subsequent start messages
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

If the worker is already started, don't assert and just return.

This fixes calling spice_server_vm_start() multiple times.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Acked-by: Frediano Ziglio <freddy77@gmail.com>
---
 server/red-worker.cpp | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/server/red-worker.cpp b/server/red-worker.cpp
index 830adc37..31a3731f 100644
--- a/server/red-worker.cpp
+++ b/server/red-worker.cpp
@@ -536,7 +536,10 @@ static void handle_dev_start(void *opaque, void *payload)
 {
     RedWorker *worker = (RedWorker*) opaque;
 
-    spice_assert(!red_qxl_is_running(worker->qxl));
+    if (red_qxl_is_running(worker->qxl)) {
+        return;
+    }
+
     if (worker->cursor_channel) {
         worker->cursor_channel->set_during_target_migrate(FALSE);
     }