Don't close handles from MsiBeginTransaction.

Assume that MsiEndTransaction must be called exactly once for each MsiBeginTransaction.

Fixes 6695

1 files changed, 3 insertions, 5 deletions

diff --git a/src/burn/engine/apply.cpp b/src/burn/engine/apply.cpp
index 4e652768..3ad22e9b 100644
--- a/src/burn/engine/apply.cpp
+++ b/src/burn/engine/apply.cpp
@@ -3123,12 +3123,10 @@ static HRESULT ExecuteMsiCommitTransaction(
         hr = MsiEngineCommitTransaction(pRollbackBoundary);
     }
 
-    if (SUCCEEDED(hr))
-    {
-        pRollbackBoundary->fActiveTransaction = FALSE;
+    // Assume that MsiEndTransaction can only be called once for each MsiBeginTransaction.
+    pRollbackBoundary->fActiveTransaction = FALSE;
 
-        ResetTransactionRegistrationState(pEngineState, TRUE);
-    }
+    ResetTransactionRegistrationState(pEngineState, TRUE);
 
 LExit:
     if (fCommitBeginCalled)