From 0e4ccb822caaa5885a6bd441b9609723c8d0ee39 Mon Sep 17 00:00:00 2001
From: Andrew Talbot <Andrew.Talbot@talbotville.com>
Date: Sat, 23 Jun 2007 17:28:50 +0100
Subject: [PATCH] msi: Fix use of uninitialized variable (Coverity).

---
 dlls/msi/action.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/msi/action.c b/dlls/msi/action.c
index ab3c914577b..820d91dcd9b 100644
--- a/dlls/msi/action.c
+++ b/dlls/msi/action.c
@@ -4648,7 +4648,7 @@ static UINT ITERATE_WriteEnvironmentString( MSIRECORD *rec, LPVOID param )
     LPWSTR deformatted, ptr;
     DWORD flags, type, size;
     LONG res;
-    HKEY env, root = HKEY_CURRENT_USER;
+    HKEY env = NULL, root = HKEY_CURRENT_USER;
 
     static const WCHAR environment[] =
         {'S','y','s','t','e','m','\\',
@@ -4759,7 +4759,7 @@ static UINT ITERATE_WriteEnvironmentString( MSIRECORD *rec, LPVOID param )
     res = RegSetValueExW(env, name, 0, type, (LPVOID)newval, size);
 
 done:
-    RegCloseKey(env);
+    if (env) RegCloseKey(env);
     msi_free(deformatted);
     msi_free(data);
     msi_free(newval);