From 27c6decae94536cae338731b6cb765aa92776486 Mon Sep 17 00:00:00 2001
From: Sean Hall <r.sean.hall@gmail.com>
Date: Tue, 4 May 2021 19:25:07 -0500
Subject: Put back the rollback cache package functionality and fix its bugs.

#3719, #4711, #5750
---
 src/burn/engine/core.cpp | 29 +++++++++++++++++++++--------
 1 file changed, 21 insertions(+), 8 deletions(-)

(limited to 'src/burn/engine/core.cpp')

diff --git a/src/burn/engine/core.cpp b/src/burn/engine/core.cpp
index 601f29a0..aab49cb7 100644
--- a/src/burn/engine/core.cpp
+++ b/src/burn/engine/core.cpp
@@ -467,6 +467,7 @@ extern "C" HRESULT CorePlan(
     pEngineState->plan.wzBundleId = pEngineState->registration.sczId;
     pEngineState->plan.wzBundleProviderKey = pEngineState->registration.sczId;
     pEngineState->plan.fDisableRollback = pEngineState->fDisableRollback;
+    pEngineState->plan.fBundleAlreadyRegistered = pEngineState->registration.fInstalled;
 
     hr = PlanSetVariables(action, &pEngineState->variables);
     ExitOnFailure(hr, "Failed to update action.");
@@ -613,6 +614,7 @@ extern "C" HRESULT CoreApply(
     BURN_APPLY_CONTEXT applyContext = { };
     BOOL fDeleteApplyCs = FALSE;
     BURN_CACHE_THREAD_CONTEXT cacheThreadContext = { };
+    BOOL fRollbackCache = FALSE;
     DWORD dwPhaseCount = 0;
     BOOTSTRAPPER_APPLYCOMPLETE_ACTION applyCompleteAction = BOOTSTRAPPER_APPLYCOMPLETE_ACTION_NONE;
 
@@ -713,6 +715,8 @@ extern "C" HRESULT CoreApply(
         applyContext.hCacheThread = ::CreateThread(NULL, 0, CacheThreadProc, &cacheThreadContext, 0, NULL);
         ExitOnNullWithLastError(applyContext.hCacheThread, hr, "Failed to create cache thread.");
 
+        fRollbackCache = TRUE;
+
         // If we're not caching in parallel, wait for the cache thread to terminate.
         if (!pEngineState->fParallelCacheAndExecute)
         {
@@ -740,23 +744,32 @@ extern "C" HRESULT CoreApply(
         }
     }
 
-    // If something went wrong or force restarted, skip cleaning.
-    if (FAILED(hr) || applyContext.fRollback || fSuspend || BOOTSTRAPPER_APPLY_RESTART_INITIATED == restart)
+    if (fSuspend || BOOTSTRAPPER_APPLY_RESTART_INITIATED == restart)
     {
-        ExitFunction();
+        // Leave cache alone.
+        fRollbackCache = FALSE;
     }
-
-    // Clean.
-    if (pEngineState->plan.cCleanActions)
+    else if (SUCCEEDED(hr))
     {
-        ApplyClean(&pEngineState->userExperience, &pEngineState->plan, pEngineState->companionConnection.hPipe);
+        // Clean.
+        fRollbackCache = FALSE;
+
+        if (pEngineState->plan.cCleanActions)
+        {
+            ApplyClean(&pEngineState->userExperience, &pEngineState->plan, pEngineState->companionConnection.hPipe);
+        }
     }
 
 LExit:
+    if (fRollbackCache)
+    {
+        ApplyCacheRollback(&pEngineState->userExperience, &pEngineState->plan, pEngineState->companionConnection.hCachePipe, &applyContext);
+    }
+
     // Unregister.
     if (fRegistered)
     {
-        ApplyUnregister(pEngineState, FAILED(hr) || applyContext.fRollback, fSuspend, restart);
+        ApplyUnregister(pEngineState, FAILED(hr), fSuspend, restart);
     }
 
     if (fElevated)
-- 
cgit v1.2.3-55-g6feb