summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorYi-Yo Chiang <yochiang@google.com>2021-02-18 06:23:52 +0000
committerAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>2021-02-18 06:23:52 +0000
commite3b8d7a0606524a882060c66fa2478811900da0a (patch)
tree6678c8767dffcae8671f2a62f86b618f43e56b5a
parent70f53ee4523964834e3e5be00d812ae991e978df (diff)
parent109b93220fb657753d902fd7f53d307c3f39f486 (diff)
downloadgsid-e3b8d7a0606524a882060c66fa2478811900da0a.tar.gz
gsid: Make createPartition() don't preempttively clear installer_ am: c4d64008ac am: 124afd8926 am: 109b93220f
Original change: https://android-review.googlesource.com/c/platform/system/gsid/+/1585026 MUST ONLY BE SUBMITTED BY AUTOMERGER Change-Id: I430e9e21b5fd5f9ffc96d25adf995e7ef683edfd
-rw-r--r--gsi_service.cpp9
1 files changed, 1 insertions, 8 deletions
diff --git a/gsi_service.cpp b/gsi_service.cpp
index 66e01ee..e4052b2 100644
--- a/gsi_service.cpp
+++ b/gsi_service.cpp
@@ -173,9 +173,6 @@ binder::Status GsiService::createPartition(const ::std::string& name, int64_t si
return binder::Status::ok();
}
- // Make sure a pending interrupted installations are cleaned up.
- installer_ = nullptr;
-
// Do some precursor validation on the arguments before diving into the
// install process.
if (size % LP_SECTOR_SIZE) {
@@ -202,11 +199,7 @@ binder::Status GsiService::createPartition(const ::std::string& name, int64_t si
installer_ = std::make_unique<PartitionInstaller>(this, install_dir_, name,
GetDsuSlot(install_dir_), size, readOnly);
progress_ = {};
- int status = installer_->StartInstall();
- if (status != INSTALL_OK) {
- installer_ = nullptr;
- }
- *_aidl_return = status;
+ *_aidl_return = installer_->StartInstall();
return binder::Status::ok();
}