Skip to content

Commit

Permalink
Merge "Revert^2 "Detect factory reset and deleteAllKeys"" into sc-dev…
Browse files Browse the repository at this point in the history
… am: 1566a5b

Original change: https://googleplex-android-review.googlesource.com/c/platform/system/vold/+/15547746

Change-Id: I89bdf3d65c2461f882d39089e64961537888955b
  • Loading branch information
ciphergoth authored and android-build-merge-worker-robot committed Aug 13, 2021
2 parents fb0d651 + 1566a5b commit eae1155
Show file tree
Hide file tree
Showing 3 changed files with 27 additions and 0 deletions.
13 changes: 13 additions & 0 deletions Keymaster.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -230,5 +230,18 @@ void Keymaster::earlyBootEnded() {
logKeystore2ExceptionIfPresent(rc, "earlyBootEnded");
}

void Keymaster::deleteAllKeys() {
::ndk::SpAIBinder binder(AServiceManager_getService(maintenance_service_name));
auto maint_service = ks2_maint::IKeystoreMaintenance::fromBinder(binder);

if (!maint_service) {
LOG(ERROR) << "Unable to connect to keystore2 maintenance service for deleteAllKeys";
return;
}

auto rc = maint_service->deleteAllKeys();
logKeystore2ExceptionIfPresent(rc, "deleteAllKeys");
}

} // namespace vold
} // namespace android
3 changes: 3 additions & 0 deletions Keymaster.h
Original file line number Diff line number Diff line change
Expand Up @@ -127,6 +127,9 @@ class Keymaster {
// be created or used.
static void earlyBootEnded();

// Tell all Keymint devices to delete all rollback-protected keys.
static void deleteAllKeys();

private:
std::shared_ptr<ks2::IKeystoreSecurityLevel> securityLevel;
DISALLOW_COPY_AND_ASSIGN(Keymaster);
Expand Down
11 changes: 11 additions & 0 deletions MetadataCrypt.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,17 @@ static bool read_key(const std::string& metadata_key_dir, const KeyGeneration& g
auto dir = metadata_key_dir + "/key";
LOG(DEBUG) << "metadata_key_dir/key: " << dir;
if (!MkdirsSync(dir, 0700)) return false;
if (!pathExists(dir)) {
auto delete_all = android::base::GetBoolProperty(
"ro.crypto.metadata_init_delete_all_keys.enabled", false);
if (delete_all) {
LOG(INFO) << "Metadata key does not exist, calling deleteAllKeys";
Keymaster::deleteAllKeys();
} else {
LOG(DEBUG) << "Metadata key does not exist but "
"ro.crypto.metadata_init_delete_all_keys.enabled is false";
}
}
auto temp = metadata_key_dir + "/tmp";
return retrieveOrGenerateKey(dir, temp, kEmptyAuthentication, gen, key);
}
Expand Down

0 comments on commit eae1155

Please sign in to comment.