[Spice-devel] [vdagent-win v2 PATCH 0/6] file_xfer fixes and some cleanups

[Christophe Fergeau]

ACK series

Christophe

On Wed, Nov 13, 2013 at 11:22:27AM +0200, Uri Lublin wrote:
> The destination directory for file_xfer was a public one (e.g. desktop
> directory of all-users). This is since vdagent.exe does not run as
> current logged in user.
> 
> This patch-set makes the creation of the file run as the logged in user,
> such that the destination directory for file-xfer is the user's
> desktop directory. Plus some cleanups.
> 
> v1->v2:
>   patch1: as_user destructor calls end() unconditionally.
>   patch3 and patch 4 are swapped
>   patch6: INC is removed -- INCLUDES added directly to AM_CPPFLAGS.
> 
> 
> Uri Lublin (6):
>   vdagent: add as_user to run tasks with user privileges
>   file_xfer: create file with user privileges
>   vdagent: file-xfer: make user desktop the target directory
>   vdagent: file_xfer: make g_key_get_string safer
>   Makefile.am: tabify
>   Makefile.am: add INCLUDES to AM_CPPFLAGS
> 
>  Makefile.am            | 14 ++++++----
>  vdagent/as_user.cpp    | 73 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  vdagent/as_user.h      | 38 ++++++++++++++++++++++++++
>  vdagent/file_xfer.cpp  | 34 ++++++++++++++++++-----
>  vdagent/file_xfer.h    |  3 ++-
>  vdagent/vdagent.vcproj |  8 ++++++
>  6 files changed, 157 insertions(+), 13 deletions(-)
>  create mode 100644 vdagent/as_user.cpp
>  create mode 100644 vdagent/as_user.h
> 
> -- 
> 1.8.3.1
> 
> _______________________________________________
> Spice-devel mailing list
> Spice-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/spice-devel/attachments/20131113/281c84e6/attachment.pgp>