The errors related to SpiceFileTransferTask can be present: * before the flush happens, on spice_file_transfer_task_read_async() * during the async flush: - cancel from user which is handled by file_xfer_flush_async - cancel/error from agent, handled on main_agent_handle_xfer_status() file_xfer_flush_finish() should not check internal errors of SpiceFileTransferTask and only be worried about errors regarding the flush itself. Note that with or without this patch, in case of errors from agent, we don't stop the flushing; it is only cancelled from user cancellation. --- src/channel-main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/channel-main.c b/src/channel-main.c index 209e08d..d071f42 100644 --- a/src/channel-main.c +++ b/src/channel-main.c @@ -1892,7 +1892,7 @@ static void file_xfer_data_flushed_cb(GObject *source_object, GError *error = NULL; file_xfer_flush_finish(channel, res, &error); - if (error || self->error) { + if (error) { spice_file_transfer_task_completed(self, error); return; } -- 2.7.4 _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel