diff options
author | Andrew Scull <ascull@google.com> | 2018-01-19 19:17:56 +0000 |
---|---|---|
committer | Andrew Scull <ascull@google.com> | 2018-01-24 18:24:38 +0000 |
commit | bd4e48cb23c2215b6017e803da9c9251ce7be650 (patch) | |
tree | a0e45e36dc297165ac758990e43ecba4bf0370b7 /authsecret/1.0/default | |
parent | 8a03c92df4685088c0a1438ee233a086dfe33ebb (diff) | |
download | platform_hardware_interfaces-bd4e48cb23c2215b6017e803da9c9251ce7be650.tar.gz platform_hardware_interfaces-bd4e48cb23c2215b6017e803da9c9251ce7be650.tar.bz2 platform_hardware_interfaces-bd4e48cb23c2215b6017e803da9c9251ce7be650.zip |
authsecret: remove factoryReset()
Factory reset will be handled by each device from recovery or in the
bootloader in response to `fastboot -w`. The requirements of the factory
reset are made explicit. As a result, VTS tests are moving to a host
side test that can factory reset between each test case.
Also clarifies when the remaining method is called.
Test: AuthSecretHidlTest
Bug: 71527305
Change-Id: I9a29568e022eb83061d8db68e1e7971fc53bd823
Diffstat (limited to 'authsecret/1.0/default')
-rw-r--r-- | authsecret/1.0/default/AuthSecret.cpp | 16 | ||||
-rw-r--r-- | authsecret/1.0/default/AuthSecret.h | 1 |
2 files changed, 6 insertions, 11 deletions
diff --git a/authsecret/1.0/default/AuthSecret.cpp b/authsecret/1.0/default/AuthSecret.cpp index 46a3ec121e..f9271e9617 100644 --- a/authsecret/1.0/default/AuthSecret.cpp +++ b/authsecret/1.0/default/AuthSecret.cpp @@ -29,16 +29,12 @@ Return<void> AuthSecret::primaryUserCredential(const hidl_vec<uint8_t>& secret) return Void(); } -Return<void> AuthSecret::factoryReset() { - // Clear all dependency on the secret. - // - // With the example of updating a security module, the stored value must be - // cleared so that the new primary user enrolled as the approver of updates. - // - // This implementation does nothing as there is no dependence on the secret. - - return Void(); -} +// Note: on factory reset, clear all dependency on the secret. +// +// With the example of updating a security module, the stored value must be +// cleared so that the new primary user enrolled as the approver of updates. +// +// This implementation does nothing as there is no dependence on the secret. } // namespace implementation } // namespace V1_0 diff --git a/authsecret/1.0/default/AuthSecret.h b/authsecret/1.0/default/AuthSecret.h index edb49b856f..387fa67008 100644 --- a/authsecret/1.0/default/AuthSecret.h +++ b/authsecret/1.0/default/AuthSecret.h @@ -22,7 +22,6 @@ using ::android::sp; struct AuthSecret : public IAuthSecret { // Methods from ::android::hardware::authsecret::V1_0::IAuthSecret follow. Return<void> primaryUserCredential(const hidl_vec<uint8_t>& secret) override; - Return<void> factoryReset() override; // Methods from ::android::hidl::base::V1_0::IBase follow. }; |