[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.

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/umpn...
==============================================================================
--- 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;