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

Commit 8c5d56ec authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Android (Google) Code Review
Browse files

Merge "Fix KM3.0 deleteKey behavior inconsistent with VTS test." into oc-dev

parents 1c6cb69c 0c3dd486
Loading
Loading
Loading
Loading
+7 −1
Original line number Diff line number Diff line
@@ -603,7 +603,13 @@ Return<ErrorCode> KeymasterDevice::deleteKey(const hidl_vec<uint8_t>& keyBlob) {
        return ErrorCode::UNIMPLEMENTED;
    }
    auto kmKeyBlob = hidlVec2KmKeyBlob(keyBlob);
    return legacy_enum_conversion(keymaster_device_->delete_key(keymaster_device_, &kmKeyBlob));
    auto rc = legacy_enum_conversion(
        keymaster_device_->delete_key(keymaster_device_, &kmKeyBlob));
    // Keymaster 3.0 requires deleteKey to return ErrorCode::OK if the key
    // blob is unusable after the call. This is equally true if the key blob was
    // unusable before.
    if (rc == ErrorCode::INVALID_KEY_BLOB) return ErrorCode::OK;
    return rc;
}

Return<ErrorCode> KeymasterDevice::deleteAllKeys() {