[ros-diffs] [cfinck] 40576: Only trigger the installer thread after we got the user token. Fixes a race condition mostly occuring under real hardware. See issue #4390 for more details.

[cfinck]

Author: cfinck
Date: Sat Apr 18 18:24:02 2009
New Revision: 40576

URL: http://svn.reactos.org/svn/reactos?rev=40576&view=rev
Log:
Only trigger the installer thread after we got the user token.
Fixes a race condition mostly occuring under real hardware.

See issue #4390 for more details.

Modified:
    trunk/reactos/base/services/umpnpmgr/umpnpmgr.c

Modified: trunk/reactos/base/services/umpnpmgr/umpnpmgr.c
URL: 
http://svn.reactos.org/svn/reactos/trunk/reactos/base/services/umpnpmgr/umpnpmgr.c?rev=40576&r1=40575&r2=40576&view=diff
==============================================================================
--- trunk/reactos/base/services/umpnpmgr/umpnpmgr.c [iso-8859-1] (original)
+++ trunk/reactos/base/services/umpnpmgr/umpnpmgr.c [iso-8859-1] Sat Apr 18 
18:24:02 2009
@@ -234,9 +234,6 @@
 
     DPRINT("PNP_ReportLogOn(%u, %u) called\n", Admin, ProcessId);
 
-    if (hInstallEvent != NULL)
-        SetEvent(hInstallEvent);
-
     /* Get the users token */
     hProcess = OpenProcess(PROCESS_ALL_ACCESS, TRUE, ProcessId);
 
@@ -259,8 +256,8 @@
     }
 
     /* Trigger the installer thread */
-    /*if (hInstallEvent != NULL)
-        SetEvent(hInstallEvent);*/
+    if (hInstallEvent)
+        SetEvent(hInstallEvent);
 
     ReturnValue = CR_SUCCESS;