summaryrefslogtreecommitdiffstats
path: root/authsecret/1.0/default
diff options
context:
space:
mode:
authorAndrew Scull <ascull@google.com>2018-01-19 19:17:56 +0000
committerAndrew Scull <ascull@google.com>2018-01-24 18:24:38 +0000
commitbd4e48cb23c2215b6017e803da9c9251ce7be650 (patch)
treea0e45e36dc297165ac758990e43ecba4bf0370b7 /authsecret/1.0/default
parent8a03c92df4685088c0a1438ee233a086dfe33ebb (diff)
downloadplatform_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.cpp16
-rw-r--r--authsecret/1.0/default/AuthSecret.h1
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.
};