Skip to content

Commit

Permalink
ext4crypt: change to upgrade key if export fails
Browse files Browse the repository at this point in the history
Add support to upgrade key when export fails with KEY_REQUIRES_UPGRADE.

Ported from
https://source.codeaurora.org/quic/la/platform/system/vold/commit/?h=LA.UM.7.9.r1-06100-sm6150.0&id=85c46eaacc60290db5e71380d89eb4d99ed67995

Change-Id: Ic64be8ade00c0b0d014370ecc9341b1ecc9b0d7a
  • Loading branch information
PeterCxy authored and mauronofrio committed Sep 18, 2019
1 parent 6971597 commit f999d65
Show file tree
Hide file tree
Showing 3 changed files with 27 additions and 9 deletions.
26 changes: 22 additions & 4 deletions crypto/ext4crypt/KeyStorage4.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -165,10 +165,28 @@ bool getEphemeralWrappedKey(km::KeyFormat format, KeyBuffer& kmKey, KeyBuffer* k
std::string key_temp;
Keymaster keymaster;
if (!keymaster) return false;
if (!keymaster.exportKey(format, kmKey, "!", "!", &key_temp)) return false;
*key = KeyBuffer(key_temp.size());
memcpy(reinterpret_cast<void*>(key->data()), key_temp.c_str(), key->size());
return true;

//Export once, if upgrade needed, upgrade and export again
bool export_again = true;
while (export_again) {
export_again = false;
auto ret = keymaster.exportKey(format, kmKey, "!", "!", &key_temp);
if (ret == km::ErrorCode::OK) {
*key = KeyBuffer(key_temp.size());
memcpy(reinterpret_cast<void*>(key->data()), key_temp.c_str(), key->size());
return true;
}
if (ret != km::ErrorCode::KEY_REQUIRES_UPGRADE) return false;
LOG(DEBUG) << "Upgrading key";
std::string kmKeyStr(reinterpret_cast<const char*>(kmKey.data()), kmKey.size());
std::string newKey;
if (!keymaster.upgradeKey(kmKeyStr, km::AuthorizationSet(), &newKey)) return false;
memcpy(reinterpret_cast<void*>(kmKey.data()), newKey.c_str(), kmKey.size());
LOG(INFO) << "Key upgraded";
export_again = true;
}
//Should never come here
return false;
}

static std::pair<km::AuthorizationSet, km::HardwareAuthToken> beginParams(
Expand Down
8 changes: 4 additions & 4 deletions crypto/ext4crypt/Keymaster4.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,7 @@ bool Keymaster::generateKey(const km::AuthorizationSet& inParams, std::string* k
return true;
}

bool Keymaster::exportKey(km::KeyFormat format, KeyBuffer& kmKey, const std::string& clientId,
km::ErrorCode Keymaster::exportKey(km::KeyFormat format, KeyBuffer& kmKey, const std::string& clientId,
const std::string& appData, std::string* key) {
auto kmKeyBlob = km::support::blob2hidlVec(std::string(kmKey.data(), kmKey.size()));
auto emptyAssign = NULL;
Expand All @@ -159,13 +159,13 @@ bool Keymaster::exportKey(km::KeyFormat format, KeyBuffer& kmKey, const std::str
auto error = mDevice->exportKey(format, kmKeyBlob, kmClientId, kmAppData, hidlCb);
if (!error.isOk()) {
LOG(ERROR) << "export_key failed: " << error.description();
return false;
return km::ErrorCode::UNKNOWN_ERROR;
}
if (km_error != km::ErrorCode::OK) {
LOG(ERROR) << "export_key failed, code " << int32_t(km_error);
return false;
return km_error;
}
return true;
return km::ErrorCode::OK;
}

bool Keymaster::deleteKey(const std::string& key) {
Expand Down
2 changes: 1 addition & 1 deletion crypto/ext4crypt/Keymaster4.h
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,7 @@ class Keymaster {
// Generate a key in the keymaster from the given params.
bool generateKey(const km::AuthorizationSet& inParams, std::string* key);
// Export a key from keymaster.
bool exportKey(km::KeyFormat format, KeyBuffer& kmKey, const std::string& clientId,
km::ErrorCode exportKey(km::KeyFormat format, KeyBuffer& kmKey, const std::string& clientId,
const std::string& appData, std::string* key);
// If the keymaster supports it, permanently delete a key.
bool deleteKey(const std::string& key);
Expand Down

0 comments on commit f999d65

Please sign in to comment.