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@xxxxxxxxxx> --- 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 _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel