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

Commit 7bed173e authored by Treehugger Robot's avatar Treehugger Robot Committed by Gerrit Code Review
Browse files

Merge "Only require RKP on T+ chipsets" into main

parents 8a218307 8be875e0
Loading
Loading
Loading
Loading
+6 −9
Original line number Original line Diff line number Diff line
@@ -1300,15 +1300,12 @@ std::pair<ErrorCode, vector<uint8_t>> KeyMintAidlTestBase::UpgradeKey(
}
}


bool KeyMintAidlTestBase::IsRkpSupportRequired() const {
bool KeyMintAidlTestBase::IsRkpSupportRequired() const {
    if (get_vsr_api_level() >= __ANDROID_API_T__) {
    // This is technically not a match to the requirements for S chipsets,
        return true;
    // however when S shipped there was a bug in the test that skipped the
    }
    // tests if KeyMint 2 was not on the system. So we allowed many chipests

    // to ship without RKP support. In T we hardened the requirements around
    if (get_vsr_api_level() >= __ANDROID_API_S__) {
    // support for RKP, so relax the test to match.
        return SecLevel() != SecurityLevel::STRONGBOX;
    return get_vsr_api_level() >= __ANDROID_API_T__;
    }

    return false;
}
}


vector<uint32_t> KeyMintAidlTestBase::ValidKeySizes(Algorithm algorithm) {
vector<uint32_t> KeyMintAidlTestBase::ValidKeySizes(Algorithm algorithm) {