According to windows vdagent fileXfer, it requires a
VD_AGENT_FILE_XFER_STATUS_CANCELLED msg when user cancel a file transfer.
Currently our code only send VD_AGENT_FILE_XFER_STATUS_ERROR, this will cause
the vdagent lock the file until it restart, you can't remove the broken file.
---
src/channel-main.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/channel-main.c b/src/channel-main.c
index 8138fd5..3c3ce5a 100644
--- a/src/channel-main.c
+++ b/src/channel-main.c
@@ -1922,7 +1922,8 @@ static void file_xfer_read_cb(GObject *source_object,
} else if (error) {
VDAgentFileXferStatusMessage msg = {
.id = self->priv->id,
- .result = VD_AGENT_FILE_XFER_STATUS_ERROR,
+ .result = error->code == G_IO_ERROR_CANCELLED?
+ VD_AGENT_FILE_XFER_STATUS_CANCELLED:
VD_AGENT_FILE_XFER_STATUS_ERROR,
};
agent_msg_queue_many(self->priv->channel, VD_AGENT_FILE_XFER_STATUS,
&msg, sizeof(msg), NULL);
--
1.9.1
On Thu, Nov 19, 2015 at 1:56 AM, Eduardo Lima (Etrunko)
<etrunko@xxxxxxxxxx> wrote:
> Hi,
>
> It seems your message was sent with HTML formatting, I can't apply it to
> the tree. Can you try re-sending it with git send-email? You can find
> the configuration settings for gmail in git send-email --help page.
>
> Regards, Eduardo.
>
> On 11/18/2015 01:29 AM, Jay.han wrote:
>>
>> According to vdagent fileXfer, it requires
>> a VD_AGENT_FILE_XFER_STATUS_CANCELLED msg when user cancel a file
>> transfer. Currently our code only send VD_AGENT_FILE_XFER_STATUS_ERROR,
>> this will cause the vdagent lock the file until it restart, you can't
>> remove the broken file.
>>
>> diff --git a/src/channel-main.c b/src/channel-main.c
>> index 8138fd5..7a9af7b 100644
>> --- a/src/channel-main.c
>> +++ b/src/channel-main.c
>> @@ -1922,7 +1922,8 @@ static void file_xfer_read_cb(GObject *source_object,
>> } else if (error) {
>> VDAgentFileXferStatusMessage msg = {
>> .id = self->priv->id,
>> - .result = VD_AGENT_FILE_XFER_STATUS_ERROR,
>> + .result = error->code == G_IO_ERROR_CANCELLED ?
>> + VD_AGENT_FILE_XFER_STATUS_CANCELLED :
>> VD_AGENT_FILE_XFER_STATUS_ERROR,
>> };
>> agent_msg_queue_many(self->priv->channel,
>> VD_AGENT_FILE_XFER_STATUS,
>> &msg, sizeof(msg), NULL);
>> --
>> Jay.Han
>>
>>
>> _______________________________________________
>> Spice-devel mailing list
>> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
>> http://lists.freedesktop.org/mailman/listinfo/spice-devel
>>
>
>
> --
> Eduardo de Barros Lima (Etrunko)
> Software Engineer - RedHat
> etrunko@xxxxxxxxxx
--
Jay.Han
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel