No need to inform of a problem under spice_file_transfer_task_completed() as the task will be finalized and we can send the error to the agent there. This change is related to split SpiceFileTransferTask from channel-main. Acked-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx> --- src/channel-main.c | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/src/channel-main.c b/src/channel-main.c index 8e8f6bd..e57d2ba 100644 --- a/src/channel-main.c +++ b/src/channel-main.c @@ -2991,16 +2991,6 @@ static void spice_file_transfer_task_completed(SpiceFileTransferTask *self, self->error = error; } - if (self->error) { - VDAgentFileXferStatusMessage msg = { - .id = self->id, - .result = self->error->code == G_IO_ERROR_CANCELLED ? - VD_AGENT_FILE_XFER_STATUS_CANCELLED : VD_AGENT_FILE_XFER_STATUS_ERROR, - }; - agent_msg_queue_many(self->channel, VD_AGENT_FILE_XFER_STATUS, - &msg, sizeof(msg), NULL); - } - if (self->pending) { /* Complete but pending is okay only if error is set */ if (self->error == NULL) { -- 2.7.4 _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel