commit 9d1460be060ddc25aacff82dc7630fa626c615d0
author Treehugger Robot <treehugger-gerrit@google.com> Fri Dec 02 20:06:50 2022 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Fri Dec 02 20:06:50 2022 +0000
tree 6c20fb079076c3889b07ffe43627cf8a6b28cda0
parent 69525ba396a1141ce420aa5cf782f1cb60f418a9
parent 9e6005ada46831b5a4cf9285bed72b1cbd4964b3

Merge "Fix nullptr access when cow reader open fails" am: 9e6005ada4

Original change: https://android-review.googlesource.com/c/platform/system/core/+/2325397

Change-Id: Ic8cc95b0548039f6c04c24e6f4209a71c878c00a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

fs_mgr/libsnapshot/snapshot_writer.cpp

diff --git a/fs_mgr/libsnapshot/snapshot_writer.cpp b/fs_mgr/libsnapshot/snapshot_writer.cpp
index 6aad3d1..82a7fd7 100644
--- a/fs_mgr/libsnapshot/snapshot_writer.cpp
+++ b/fs_mgr/libsnapshot/snapshot_writer.cpp
@@ -93,6 +93,9 @@
 
 std::unique_ptr<FileDescriptor> CompressedSnapshotWriter::OpenReader() {
     auto cow = OpenCowReader();
+    if (cow == nullptr) {
+        return nullptr;
+    }
 
     auto reader = std::make_unique<CompressedSnapshotReader>();
     if (!reader->SetCow(std::move(cow))) {