Re: [PATCH] vdagent-win: Assure you set a new HKEY_CURRENT_USER changing user.

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

 



On Thu, Apr 23, 2015 at 09:09:05AM -0400, Frediano Ziglio wrote:
> During impersonation one thing that Windows does is to try to create HKEY_CURRENT_USER. This is done automatically for you by Windows when it launch a process.
> HKEY_CURRENT_KEY is a predefined key value.
> Anyway when you impersonate a user HKEY_CURRENT_USER is pointed to HKEY_USERS\<sid-string-of-user> (think as a symbolic link). If HEKY_CURRENT_USER is already opened (which usually is the case) Windows do not change the registry key and you will still find old values.
> Probably you should close the key even before RevertToSelf. Surely IMHO before ImpersonateLoggedOnUser.

What I'm uncomfortable with is to close something I don't think we own.
I would not free() memory I don't own, nor close() a file descriptor I
don't own. Are things different with these registry key handles, and is
it allowed and typical to close HKEY_CURRENT_USER?

Christophe

Attachment: pgp6HwQtWLCuw.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]