Re: [patch]spice-gtk: encoding problem when drag a file from client to guest

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

----- Original Message -----
> hi, here's my environment:
> client: win7(default encoding is GBK)
> guest the same with client
> spice-gtk-0.23d
> 
> a file in guest with a name of Chinese
> the file name is wrong when drag it to guest desktop
> (I think Japanese and Korean have the same problem )
> 
> the following is my patch to " spice-gtk-0.23/gtk/channel-main.c"
> 
> Regards
> 
> --- spice-gtk-0.23/gtk/channel-main.c 2014-02-06 19:08:07.000000000 +0800
> +++ spice-gtk-0.23_modified/gtk/channel-main.c 2014-02-17 20:12:55.037816310
> +0800
> @@ -2786,8 +2786,10 @@
> 
> /* File name */
> basename = g_file_get_basename(file);
> - g_key_file_set_string(keyfile, "vdagent-file-xfer", "name", basename);

>From glib API reference, I understand basename is a byte string of unspecified encoding.

> + gchar *basename_loc = g_locale_from_utf8(basename, -1, NULL, NULL, NULL);

So this is probably wrong. You should also check for errors.

Instead, I think we should use g_file_info_get_display_name()

> + g_key_file_set_string(keyfile, "vdagent-file-xfer", "name", basename_loc);
> g_free(basename);
> + g_free(basename_loc);
> /* File size */
> g_key_file_set_uint64(keyfile, "vdagent-file-xfer", "size", task->file_size);
> 

Can you send patches using git-send-email (inline by default) or as attachments?

thanks
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]