diff options
author | Hasini Gunasinghe <hasinitg@google.com> | 2021-08-02 22:28:39 +0000 |
---|---|---|
committer | Hasini Gunasinghe <hasinitg@google.com> | 2021-08-05 20:04:01 +0000 |
commit | ac057d3a9945bf2f0c7ce3ad06eb969d67db0d17 (patch) | |
tree | 1999a302cc1601f63d04f6aac81d703c805ecab2 | |
parent | 301d13c4ce39390538d2b604b7d72f858a112252 (diff) | |
download | security-ac057d3a9945bf2f0c7ce3ad06eb969d67db0d17.tar.gz |
Log metrics for RKP fallback event.
This CL adds metrics logging for fallback during RKP hybrid mode.
Ignore-AOSP-First: No merge path from AOSP.
Test: atest com.android.remoteprovisioner.unittest.ServerToSystemTest#testFallback
on cuttlefish, followed by ./statsd_testdrive 10124
Bug: 194510784
Merged-In: I3d6184df02828fef1d0110688afe1ba2db10eae9
Change-Id: I3d6184df02828fef1d0110688afe1ba2db10eae9
-rw-r--r-- | keystore2/src/remote_provisioning.rs | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/keystore2/src/remote_provisioning.rs b/keystore2/src/remote_provisioning.rs index 12364317..212bf399 100644 --- a/keystore2/src/remote_provisioning.rs +++ b/keystore2/src/remote_provisioning.rs @@ -43,7 +43,9 @@ use std::sync::atomic::{AtomicBool, Ordering}; use crate::database::{CertificateChain, KeystoreDB, Uuid}; use crate::error::{self, map_or_log_err, map_rem_prov_error, Error}; use crate::globals::{get_keymint_device, get_remotely_provisioned_component, DB}; +use crate::metrics_store::log_rkp_error_stats; use crate::utils::{watchdog as wd, Asp}; +use android_security_metrics::aidl::android::security::metrics::RkpError::RkpError as MetricsRkpError; /// Contains helper functions to check if remote provisioning is enabled on the system and, if so, /// to assign and retrieve attestation keys and certificate chains. @@ -189,6 +191,7 @@ impl RemProvState { ), e ); + log_rkp_error_stats(MetricsRkpError::FALL_BACK_DURING_HYBRID); Ok(None) } Ok(v) => match v { |