From 5bce4dda0c01d57edb403077a092e9fc7d8b4efb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Herv=C3=A9=20Poussineau?= <hpoussin@reactos.org>
Date: Wed, 24 Oct 2007 07:49:40 +0000
Subject: [PATCH] Better check of GetWindowsDirectory() return value

svn path=/trunk/; revision=29846
---
 reactos/dll/win32/msgina/msgina.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/reactos/dll/win32/msgina/msgina.c b/reactos/dll/win32/msgina/msgina.c
index f843a78f590..8b1a24aff61 100644
--- a/reactos/dll/win32/msgina/msgina.c
+++ b/reactos/dll/win32/msgina/msgina.c
@@ -207,6 +207,13 @@ WlxStartApplication(
 	UINT len;
 	BOOL ret;
 
+	len = GetWindowsDirectoryW(CurrentDirectory, MAX_PATH);
+	if (len == 0 || len > MAX_PATH)
+	{
+		WARN("GetWindowsDirectoryW() failed\n");
+		return FALSE;
+	}
+
 	ret = DuplicateTokenEx(pgContext->UserToken, MAXIMUM_ALLOWED, NULL, SecurityImpersonation, TokenPrimary, &hAppToken);
 	if (!ret)
 	{
@@ -222,13 +229,6 @@ WlxStartApplication(
 	StartupInfo.wShowWindow = SW_SHOW;
 	StartupInfo.lpDesktop = pszDesktopName;
 
-	len = GetWindowsDirectoryW(CurrentDirectory, MAX_PATH);
-	if (len > MAX_PATH)
-	{
-		WARN("GetWindowsDirectoryW() failed\n");
-		CloseHandle(hAppToken);
-		return FALSE;
-	}
 	ret = CreateProcessAsUserW(
 		hAppToken,
 		pszCmdLine,