From 5891ab2bfd8bc3d91c4e9d8fc30d27393d9c60b1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Herm=C3=A8s=20B=C3=A9lusca-Ma=C3=AFto?=
 <hermes.belusca-maito@reactos.org>
Date: Tue, 19 Oct 2021 23:09:16 +0200
Subject: [PATCH] [WIN32SS:NTUSER] Make sure to hold User Global Lock before
 manipulating desktops. (#4053)

---
 win32ss/user/ntuser/desktop.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/win32ss/user/ntuser/desktop.c b/win32ss/user/ntuser/desktop.c
index 84cac5fb47b..4d8f2956b44 100644
--- a/win32ss/user/ntuser/desktop.c
+++ b/win32ss/user/ntuser/desktop.c
@@ -561,6 +561,8 @@ IntResolveDesktop(
     BOOLEAN bInteractive = FALSE;
     BOOLEAN bAccessAllowed = FALSE;
 
+    ASSERT(UserIsEnteredExclusive());
+
     ASSERT(phWinSta);
     ASSERT(phDesktop);
     ASSERT(DesktopPath);
@@ -2329,6 +2331,8 @@ IntCreateDesktop(
 
     TRACE("Enter IntCreateDesktop\n");
 
+    ASSERT(UserIsEnteredExclusive());
+
     ASSERT(phDesktop);
     *phDesktop = NULL;
 
@@ -2808,7 +2812,7 @@ NtUserResolveDesktop(
     if (!NT_SUCCESS(Status))
         return NULL;
 
-    // UserEnterShared();
+    UserEnterExclusive();
 
     _SEH2_TRY
     {
@@ -2863,7 +2867,7 @@ NtUserResolveDesktop(
     ReleaseCapturedUnicodeString(&CapturedDesktopPath, UserMode);
 
 Quit:
-    // UserLeave();
+    UserLeave();
 
     /* Dereference the process object */
     ObDereferenceObject(Process);