Evolution-x/hardware_interfaces
1
0
Fork 0
mirror of https://github.com/Evolution-X/hardware_interfaces synced 2026-02-02 13:49:45 +00:00
Code Issues Packages Projects Releases Wiki Activity

Read VSR level from correct property. am: 35db349596

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2128833

Change-Id: I856ff506c20cbfe2f926f5f5e737426524e055bc
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Shawn Willden
2022-06-20 01:38:43 +00:00
committed by Automerger Merge Worker
parent 2b403196c9 35db349596
commit 4e97ec021e
2 changed files with 36 additions and 30 deletions
Download Patch File Download Diff File Expand all files Collapse all files

33
keymaster/4.0/vts/functional/keymaster_hidl_hal_test.cpp
View File

@@ -388,25 +388,28 @@ bool avb_verification_enabled() {
} }
int get_vsr_api_level() { int get_vsr_api_level() {
int api_level = ::android::base::GetIntProperty("ro.board.api_level", -1); int vendor_api_level = ::android::base::GetIntProperty("ro.vendor.api_level", -1);
if (api_level == -1) { if (vendor_api_level != -1) {
api_level = ::android::base::GetIntProperty("ro.board.first_api_level", -1); return vendor_api_level;
} }
if (api_level == -1) {
api_level = ::android::base::GetIntProperty("ro.vndk.version", -1); // Android S and older devices do not define ro.vendor.api_level
vendor_api_level = ::android::base::GetIntProperty("ro.board.api_level", -1);
if (vendor_api_level == -1) {
vendor_api_level = ::android::base::GetIntProperty("ro.board.first_api_level", -1);
} }
// We really should have a VSR API level by now. But on cuttlefish, and perhaps other weird
// devices, we may not. So, we use the SDK first or current API level if needed. If this goes int product_api_level = ::android::base::GetIntProperty("ro.product.first_api_level", -1);
// wrong, it should go wrong in the direction of being too strict rather than too lenient, which if (product_api_level == -1) {
// should provoke someone to examine why we don't have proper VSR API level properties. product_api_level = ::android::base::GetIntProperty("ro.build.version.sdk", -1);
if (api_level == -1) { EXPECT_NE(product_api_level, -1) << "Could not find ro.build.version.sdk";
api_level = ::android::base::GetIntProperty("ro.product.first_api_level", -1);
} }
if (api_level == -1) {
api_level = ::android::base::GetIntProperty("ro.build.version.sdk", -1); // VSR API level is the minimum of vendor_api_level and product_api_level.
if (vendor_api_level == -1 || vendor_api_level > product_api_level) {
return product_api_level;
} }
EXPECT_NE(api_level, -1) << "Could not find a VSR level, or equivalent."; return vendor_api_level;
return api_level;
} }
bool is_gsi() { bool is_gsi() {

33
security/keymint/aidl/vts/functional/KeyMintAidlTestBase.cpp
View File

@@ -1461,25 +1461,28 @@ void verify_subject(const X509* cert, //
} }
int get_vsr_api_level() { int get_vsr_api_level() {
int api_level = ::android::base::GetIntProperty("ro.board.api_level", -1); int vendor_api_level = ::android::base::GetIntProperty("ro.vendor.api_level", -1);
if (api_level == -1) { if (vendor_api_level != -1) {
api_level = ::android::base::GetIntProperty("ro.board.first_api_level", -1); return vendor_api_level;
} }
if (api_level == -1) {
api_level = ::android::base::GetIntProperty("ro.vndk.version", -1); // Android S and older devices do not define ro.vendor.api_level
vendor_api_level = ::android::base::GetIntProperty("ro.board.api_level", -1);
if (vendor_api_level == -1) {
vendor_api_level = ::android::base::GetIntProperty("ro.board.first_api_level", -1);
} }
// We really should have a VSR API level by now. But on cuttlefish, and perhaps other weird
// devices, we may not. So, we use the SDK first or current API level if needed. If this goes int product_api_level = ::android::base::GetIntProperty("ro.product.first_api_level", -1);
// wrong, it should go wrong in the direction of being too strict rather than too lenient, which if (product_api_level == -1) {
// should provoke someone to examine why we don't have proper VSR API level properties. product_api_level = ::android::base::GetIntProperty("ro.build.version.sdk", -1);
if (api_level == -1) { EXPECT_NE(product_api_level, -1) << "Could not find ro.build.version.sdk";
api_level = ::android::base::GetIntProperty("ro.product.first_api_level", -1);
} }
if (api_level == -1) {
api_level = ::android::base::GetIntProperty("ro.build.version.sdk", -1); // VSR API level is the minimum of vendor_api_level and product_api_level.
if (vendor_api_level == -1 || vendor_api_level > product_api_level) {
return product_api_level;
} }
EXPECT_NE(api_level, -1) << "Could not find a VSR level, or equivalent."; return vendor_api_level;
return api_level;
} }
bool is_gsi_image() { bool is_gsi_image() {