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
> 

Perhaps this should be the solution (have to test it):

https://msdn.microsoft.com/en-us/library/windows/desktop/ms724854%28v=vs.85%29.aspx

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