Re: [vdagent-win v2 PATCH 0/6] file_xfer fixes and some cleanups

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

 



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@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/spice-devel

Attachment: pgpUYLUvMTyAz.pgp
Description: PGP signature

_______________________________________________
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]