Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d422b570 authored by Akilesh Kailash's avatar Akilesh Kailash Committed by Automerger Merge Worker
Browse files

Merge "libsnapshot: Error handling after QuerySnapshotStatus" am: 6f967229 am: 0725099e 

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

Change-Id: I370e8ede6bfcc83280dad8bc41926c16c2ea372d
parents 43949b2a 0725099e

fs_mgr/libsnapshot/snapshot.cpp

+11 −0
Original line number Diff line number Diff line
@@ -518,6 +518,13 @@ bool SnapshotManager::MapSnapshot(LockedFile* lock, const std::string& name, 
            break;

    }



    if (mode == SnapshotStorageMode::Persistent && status.state() == SnapshotState::MERGING) {

        LOG(ERROR) << "Snapshot: " << name

                   << " has snapshot status Merging but mode set to Persistent."

                   << " Changing mode to Snapshot-Merge.";

        mode = SnapshotStorageMode::Merge;

    }



    DmTable table;

    table.Emplace<DmTargetSnapshot>(0, snapshot_sectors, base_device, cow_device, mode,

                                    kSnapshotChunkSize);
@@ -886,6 +893,10 @@ bool SnapshotManager::QuerySnapshotStatus(const std::string& dm_name, std::strin 
    if (target_type) {

        *target_type = DeviceMapper::GetTargetType(target.spec);

    }

    if (!status->error.empty()) {

        LOG(ERROR) << "Snapshot: " << dm_name << " returned error code: " << status->error;

        return false;

    }

    return true;

}