[Spice-devel] [PATCH spice-gtk v6 17/18] test-cd-emu: Make sure we can call spice_usb_backend_channel_flush_writes

Frediano Ziglio fziglio at redhat.com
Thu Sep 5 13:42:40 UTC 2019


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 <fziglio at 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 fb023a99..690423bb 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.20.1



More information about the Spice-devel mailing list