From 88045aef5999ab813f0ee22ca67429873954634d Mon Sep 17 00:00:00 2001 From: David Anderson Date: Mon, 4 Feb 2019 19:02:19 -0800 Subject: [PATCH] Enable metadata encryption for userdata_gsi. Rather than constructing a userdata fstab entry from scratch, this patch will modify the vendor fstab to preserve the desired encryption properties and filesystem type. Bug: 123906417 Test: manual test Change-Id: I338715fc62628169e8eafbf4a3125e4aadf0ff15 --- fs_mgr/fs_mgr_fstab.cpp | 41 +++++++++++++++++++++++++++++------------ 1 file changed, 29 insertions(+), 12 deletions(-) diff --git a/fs_mgr/fs_mgr_fstab.cpp b/fs_mgr/fs_mgr_fstab.cpp index 9a8ad5692..6d86bed73 100644 --- a/fs_mgr/fs_mgr_fstab.cpp +++ b/fs_mgr/fs_mgr_fstab.cpp @@ -587,18 +587,7 @@ std::set ExtraBootDevices(const Fstab& fstab) { return boot_devices; } -void EraseFstabEntry(Fstab* fstab, const std::string& mount_point) { - auto iter = std::remove_if(fstab->begin(), fstab->end(), - [&](const auto& entry) { return entry.mount_point == mount_point; }); - fstab->erase(iter, fstab->end()); -} - -void TransformFstabForGsi(Fstab* fstab) { - EraseFstabEntry(fstab, "/system"); - EraseFstabEntry(fstab, "/data"); - - fstab->emplace_back(BuildGsiSystemFstabEntry()); - +FstabEntry BuildGsiUserdataFstabEntry() { constexpr uint32_t kFlags = MS_NOATIME | MS_NOSUID | MS_NODEV; FstabEntry userdata = { @@ -614,6 +603,34 @@ void TransformFstabForGsi(Fstab* fstab) { userdata.fs_mgr_flags.quota = true; userdata.fs_mgr_flags.late_mount = true; userdata.fs_mgr_flags.formattable = true; + return userdata; +} + +void EraseFstabEntry(Fstab* fstab, const std::string& mount_point) { + auto iter = std::remove_if(fstab->begin(), fstab->end(), + [&](const auto& entry) { return entry.mount_point == mount_point; }); + fstab->erase(iter, fstab->end()); +} + +void TransformFstabForGsi(Fstab* fstab) { + // Inherit fstab properties for userdata. + FstabEntry userdata; + if (FstabEntry* entry = GetEntryForMountPoint(fstab, "/data")) { + userdata = *entry; + userdata.blk_device = "userdata_gsi"; + userdata.fs_mgr_flags.logical = true; + userdata.fs_mgr_flags.formattable = true; + if (!userdata.key_dir.empty()) { + userdata.key_dir += "/gsi"; + } + } else { + userdata = BuildGsiUserdataFstabEntry(); + } + + EraseFstabEntry(fstab, "/system"); + EraseFstabEntry(fstab, "/data"); + + fstab->emplace_back(BuildGsiSystemFstabEntry()); fstab->emplace_back(userdata); }