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

Frediano Ziglio fziglio at redhat.com
Wed Aug 28 14:14:20 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 138dc64d..27ad8ef6 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