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

Commit 6f967229 authored by Akilesh Kailash's avatar Akilesh Kailash Committed by Gerrit Code Review
Browse files

Merge "libsnapshot: Error handling after QuerySnapshotStatus"

parents c92ae83a 3ad55a96

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;

}