commit d66ecf171c91a456d84d8da086530dd37a31066a
author Yifan Hong <elsk@google.com> Tue Feb 04 11:15:50 2020 -0800
committer Yifan Hong <elsk@google.com> Tue Feb 04 11:15:51 2020 -0800
tree f86c3696286ecb813023ed898e340e5bcb240f61
parent bae2784f42f598d57b0155cc9336a73782c8def0

Fix nullptr segv

snapshot_ is only initialized when VAB feature
flag is set.

Fixes: 148771898
Test: TH
Change-Id: I5552cf19213820c267cb5f48cd923144474a22a1

dynamic_partition_control_android.cc

diff --git a/dynamic_partition_control_android.cc b/dynamic_partition_control_android.cc
index 072a3ec..938a75b 100644
--- a/dynamic_partition_control_android.cc
+++ b/dynamic_partition_control_android.cc
@@ -621,7 +621,8 @@
 }
 
 bool DynamicPartitionControlAndroid::FinishUpdate() {
-  if (snapshot_->GetUpdateState() == UpdateState::Initiated) {
+  if (GetVirtualAbFeatureFlag().IsEnabled() &&
+      snapshot_->GetUpdateState() == UpdateState::Initiated) {
     LOG(INFO) << "Snapshot writes are done.";
     return snapshot_->FinishedSnapshotWrites();
   }