Fix a leaking handle.
Make MsiCollectUserInfo more internally consistent.
Thanks to Mike McCormack.

dlls/msi/msi.c

diff --git a/dlls/msi/msi.c b/dlls/msi/msi.c
index f2b99b1..40f6f9d 100644
--- a/dlls/msi/msi.c
+++ b/dlls/msi/msi.c
@@ -36,6 +36,7 @@
 #include "winver.h"
 #include "winuser.h"
 #include "wine/unicode.h"
+#include "action.h"
 
 WINE_DEFAULT_DEBUG_CHANNEL(msi);
 
@@ -1604,6 +1605,7 @@
     if (pcchSerialBuf)
         *pcchSerialBuf = sz / sizeof(WCHAR);
     
+    RegCloseKey(hkey);
     return USERINFOSTATE_PRESENT;
 }
 
@@ -1622,6 +1624,7 @@
 {
     MSIHANDLE handle;
     UINT rc;
+    MSIPACKAGE *package;
     static const WCHAR szFirstRun[] = {'F','i','r','s','t','R','u','n',0};
 
     TRACE("(%s)\n",debugstr_w(szProduct));
@@ -1630,7 +1633,10 @@
     if (rc != ERROR_SUCCESS)
         return ERROR_INVALID_PARAMETER;
 
-    rc = MsiDoActionW(handle, szFirstRun);
+    package = msihandle2msiinfo(handle, MSIHANDLETYPE_PACKAGE);
+    rc = ACTION_PerformUIAction(package, szFirstRun);
+    msiobj_release( &package->hdr );
+
     MsiCloseHandle(handle);
 
     return rc;
@@ -1640,6 +1646,7 @@
 {
     MSIHANDLE handle;
     UINT rc;
+    MSIPACKAGE *package;
     static const WCHAR szFirstRun[] = {'F','i','r','s','t','R','u','n',0};
 
     TRACE("(%s)\n",debugstr_a(szProduct));
@@ -1648,7 +1655,10 @@
     if (rc != ERROR_SUCCESS)
         return ERROR_INVALID_PARAMETER;
 
-    rc = MsiDoActionW(handle, szFirstRun);
+    package = msihandle2msiinfo(handle, MSIHANDLETYPE_PACKAGE);
+    rc = ACTION_PerformUIAction(package, szFirstRun);
+    msiobj_release( &package->hdr );
+
     MsiCloseHandle(handle);
 
     return rc;