From: Frediano Ziglio <fzig...@redhat.com>

Currently we call this function when the SPICE channel is up
however this function should continue to work as in theory
the channel could avoid to handle the message and stop the flow
(for instance to implement some kind of flow limitation)
and so will need to call this function again.
This was failing in the first USB emulation implementation
causing a crash.

Signed-off-by: Frediano Ziglio <fzig...@redhat.com>
---
 tests/cd-emu.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/tests/cd-emu.c b/tests/cd-emu.c
index fb023a9..690423b 100644
--- a/tests/cd-emu.c
+++ b/tests/cd-emu.c
@@ -164,6 +164,10 @@ device_iteration(const int loop, const bool 
attach_on_connect)
     g_assert_cmpint(hellos_sent, ==, hellos_expected);
     g_assert_cmpint(messages_sent, >=, messages_expected);
 
+    spice_usb_backend_channel_flush_writes(usb_ch);
+    g_assert_cmpint(hellos_sent, ==, hellos_expected);
+    g_assert_cmpint(messages_sent, >=, messages_expected);
+
     // send hello reply
     if (loop == 0) {
         DATA_START
-- 
2.21.0

_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to