[Spice-devel] [PATCH 17/18] display: replace some dubious asserts

Frediano Ziglio fziglio at redhat.com
Fri Nov 20 03:17:41 PST 2015


From: Marc-André Lureau <marcandre.lureau at gmail.com>

---
 server/display-channel.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/server/display-channel.c b/server/display-channel.c
index 381a582..7aa3ea9 100644
--- a/server/display-channel.c
+++ b/server/display-channel.c
@@ -860,10 +860,10 @@ int display_channel_wait_for_migrate_data(DisplayChannel *display)
     RedChannelClient *rcc;
 
     spice_debug(NULL);
-    spice_assert(channel->clients_num == 1);
+    spice_warn_if_fail(channel->clients_num == 1);
 
     rcc = SPICE_CONTAINEROF(ring_get_head(&channel->clients), RedChannelClient, channel_link);
-    spice_assert(red_channel_client_waits_for_migrate_data(rcc));
+    spice_return_val_if_fail(red_channel_client_waits_for_migrate_data(rcc), FALSE);
 
     for (;;) {
         red_channel_client_receive(rcc);
-- 
2.4.3



More information about the Spice-devel mailing list