From 0d914d8c2a4621638d0b5bd7748b1f935ae6136a Mon Sep 17 00:00:00 2001 From: Jack Yu Date: Tue, 17 Dec 2019 16:11:40 +0800 Subject: [PATCH] Add android.hardware.secure_element@1.2 Test: None Bug: 142495673 Change-Id: Ie0f3e87d58047b76ea47990a49d3eac745967255 --- .../compatibility_matrix.current.xml | 2 +- secure_element/1.2/Android.bp | 18 ++++ secure_element/1.2/ISecureElement.hal | 35 ++++++ secure_element/1.2/vts/functional/Android.bp | 27 +++++ .../VtsHalSecureElementV1_2TargetTest.cpp | 101 ++++++++++++++++++ 5 files changed, 182 insertions(+), 1 deletion(-) create mode 100644 secure_element/1.2/Android.bp create mode 100644 secure_element/1.2/ISecureElement.hal create mode 100644 secure_element/1.2/vts/functional/Android.bp create mode 100644 secure_element/1.2/vts/functional/VtsHalSecureElementV1_2TargetTest.cpp diff --git a/compatibility_matrices/compatibility_matrix.current.xml b/compatibility_matrices/compatibility_matrix.current.xml index 9e29ec7749..98068985b5 100644 --- a/compatibility_matrices/compatibility_matrix.current.xml +++ b/compatibility_matrices/compatibility_matrix.current.xml @@ -382,7 +382,7 @@ android.hardware.secure_element - 1.0 + 1.0-2 ISecureElement eSE[1-9][0-9]* diff --git a/secure_element/1.2/Android.bp b/secure_element/1.2/Android.bp new file mode 100644 index 0000000000..e134771880 --- /dev/null +++ b/secure_element/1.2/Android.bp @@ -0,0 +1,18 @@ +// This file is autogenerated by hidl-gen -Landroidbp. + +hidl_interface { + name: "android.hardware.secure_element@1.2", + root: "android.hardware", + vndk: { + enabled: true, + }, + srcs: [ + "ISecureElement.hal", + ], + interfaces: [ + "android.hardware.secure_element@1.0", + "android.hardware.secure_element@1.1", + "android.hidl.base@1.0", + ], + gen_java: true, +} diff --git a/secure_element/1.2/ISecureElement.hal b/secure_element/1.2/ISecureElement.hal new file mode 100644 index 0000000000..16cc5775b5 --- /dev/null +++ b/secure_element/1.2/ISecureElement.hal @@ -0,0 +1,35 @@ +/* + * Copyright (C) 2019 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package android.hardware.secure_element@1.2; + +import @1.1::ISecureElementHalCallback; +import @1.1::ISecureElement; +import @1.0::SecureElementStatus; + +interface ISecureElement extends @1.1::ISecureElement { + /** + * Reset the Secure Element. + * + * HAL should trigger reset to the secure element. It could hardware power cycle or + * a soft reset depends on hardware design. + * HAL service must send onStateChange() with connected equal to true + * after resetting and all the re-initialization has been successfully completed. + * + * @return SecureElementStatus::SUCCESS on success and SecureElementStatus::FAILED on error. + */ + reset() generates (SecureElementStatus status); +}; diff --git a/secure_element/1.2/vts/functional/Android.bp b/secure_element/1.2/vts/functional/Android.bp new file mode 100644 index 0000000000..a1732108ab --- /dev/null +++ b/secure_element/1.2/vts/functional/Android.bp @@ -0,0 +1,27 @@ +// +// Copyright (C) 2019 The Android Open Source Project +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. +// + +cc_test { + name: "VtsHalSecureElementV1_2TargetTest", + defaults: ["VtsHalTargetTestDefaults"], + srcs: ["VtsHalSecureElementV1_2TargetTest.cpp"], + static_libs: [ + "android.hardware.secure_element@1.0", + "android.hardware.secure_element@1.1", + "android.hardware.secure_element@1.2", + ], + test_suites: ["general-tests", "vts-core"], +} diff --git a/secure_element/1.2/vts/functional/VtsHalSecureElementV1_2TargetTest.cpp b/secure_element/1.2/vts/functional/VtsHalSecureElementV1_2TargetTest.cpp new file mode 100644 index 0000000000..98e45021d9 --- /dev/null +++ b/secure_element/1.2/vts/functional/VtsHalSecureElementV1_2TargetTest.cpp @@ -0,0 +1,101 @@ +/* + * Copyright (C) 2019 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +#include + +#define LOG_TAG "secure_element_hidl_hal_test" +#include + +#include +#include +#include +#include +#include +#include + +#include + +using ::android::sp; +using ::android::hardware::hidl_string; +using ::android::hardware::Return; +using ::android::hardware::Void; +using ::android::hardware::secure_element::V1_0::SecureElementStatus; +using ::android::hardware::secure_element::V1_1::ISecureElementHalCallback; +using ::android::hardware::secure_element::V1_2::ISecureElement; + +constexpr char kCallbackNameOnStateChange[] = "onStateChange"; + +class SecureElementCallbackArgs { + public: + bool state_; + hidl_string reason_; +}; + +class SecureElementHalCallback + : public ::testing::VtsHalHidlTargetCallbackBase, + public ISecureElementHalCallback { + public: + virtual ~SecureElementHalCallback() = default; + + Return onStateChange_1_1(bool state, const hidl_string& reason) override { + SecureElementCallbackArgs args; + args.state_ = state; + args.reason_ = reason; + NotifyFromCallback(kCallbackNameOnStateChange, args); + return Void(); + }; + + Return onStateChange(__attribute__((unused)) bool state) override { return Void(); } +}; + +class SecureElementHidlTest : public ::testing::TestWithParam { + public: + virtual void SetUp() override { + LOG(INFO) << "get service with name:" << GetParam(); + se_ = ISecureElement::getService(GetParam()); + ASSERT_NE(se_, nullptr); + + se_cb_ = new SecureElementHalCallback(); + ASSERT_NE(se_cb_, nullptr); + se_->init_1_1(se_cb_); + auto res = se_cb_->WaitForCallback(kCallbackNameOnStateChange); + EXPECT_TRUE(res.no_timeout); + EXPECT_TRUE(res.args->state_); + EXPECT_NE(res.args->reason_, ""); + } + + sp se_; + sp se_cb_; +}; + +/* + * Reset: + * Calls reset() + * Checks status + * Check onStateChange is received with connected state set to true + */ +TEST_P(SecureElementHidlTest, Reset) { + EXPECT_EQ(SecureElementStatus::SUCCESS, se_->reset()); + + auto res = se_cb_->WaitForCallback(kCallbackNameOnStateChange); + EXPECT_TRUE(res.no_timeout); + EXPECT_TRUE(res.args->state_); +} + +INSTANTIATE_TEST_SUITE_P( + PerInstance, SecureElementHidlTest, + testing::ValuesIn(android::hardware::getAllHalInstanceNames(ISecureElement::descriptor)), + android::hardware::PrintInstanceNameToString);