commit 55c2bfe0f7ed671b8e1b59b38e1e485a1e937886
author Yifan Hong <elsk@google.com> Mon Jan 13 17:01:19 2020 -0800
committer Yifan Hong <elsk@google.com> Thu Jan 16 05:05:48 2020 +0000
tree 99c3e54dd7477ee5c834707760f5ef95deed21a5
parent 6561303cc3020e1049e1fd67f999d19097e72ede

DeltaPerformer: Don't destroy previously allocated space

After allocateSpaceForPayload() is called, applyPayload() with
the same hash should not destroy the allocated files then re-allocate.

Fix it so that DeltaPerformer::ResetUpdateProgress skip resetting
the hash in this case.

Bug: 138808058
Test: update_engine_client --allocate, then --apply

Change-Id: Ibc2a7449a6953a707d1c4f23ee11d572f498457c

payload_consumer/delta_performer.cc

diff --git a/payload_consumer/delta_performer.cc b/payload_consumer/delta_performer.cc
index bb7c98c..d8e58b4 100644
--- a/payload_consumer/delta_performer.cc
+++ b/payload_consumer/delta_performer.cc
@@ -1959,7 +1959,10 @@
   return true;
 }
 
-bool DeltaPerformer::ResetUpdateProgress(PrefsInterface* prefs, bool quick) {
+bool DeltaPerformer::ResetUpdateProgress(
+    PrefsInterface* prefs,
+    bool quick,
+    bool skip_dynamic_partititon_metadata_updated) {
   TEST_AND_RETURN_FALSE(prefs->SetInt64(kPrefsUpdateStateNextOperation,
                                         kUpdateStateOperationInvalid));
   if (!quick) {
@@ -1973,7 +1976,11 @@
     prefs->SetInt64(kPrefsResumedUpdateFailures, 0);
     prefs->Delete(kPrefsPostInstallSucceeded);
     prefs->Delete(kPrefsVerityWritten);
-    prefs->Delete(kPrefsDynamicPartitionMetadataUpdated);
+
+    if (!skip_dynamic_partititon_metadata_updated) {
+      LOG(INFO) << "Resetting recorded hash for prepared partitions.";
+      prefs->Delete(kPrefsDynamicPartitionMetadataUpdated);
+    }
   }
   return true;
 }