Intravision/reactos
1
0
Fork 0
mirror of https://github.com/reactos/reactos.git synced 2025-04-04 20:50:41 +00:00
Code Issues Projects Releases Packages Wiki Activity

[WINLOGON] Let Winlogon assign security to desktop when a user logs in now

Browse source
This commit is contained in:
George Bișoc 2022-05-08 00:51:54 +02:00
parent aa815e1cfa
commit 07227562c0
No known key found for this signature in database
GPG key ID: 688C4FBE25D7DEF6
1 changed files with 0 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
base/system/winlogon/security.c
View file

@ -1446,21 +1446,12 @@ AllowAccessOnSession(
goto Quit;
}
/*
* FIXME: Desktop security management is broken. The application desktop gets created
* with CreateDesktopW() API call with an initial and defined security descriptor for it yet
* when we are giving access to the logged in user, SetUserObjectSecurity() API call fails to set
* new security because the desktop in question has no prior security descriptor (when it's
* been assigned even before!!!). This chunk of code must be enabled when this gets fixed.
*/
#if 0
/* Allow application desktop access to this user within this session */
if (!AllowDesktopAccessToUser(Session->ApplicationDesktop, LogonSid))
{
ERR("AllowAccessOnSession(): Failed to allow application desktop access to the logon user!\n");
goto Quit;
}
#endif
/* Get the length of this logon SID */
SidLength = GetLengthSid(LogonSid);