diff options
author | Shaquille Johnson <ssjohnson@google.com> | 2024-02-28 22:14:05 +0000 |
---|---|---|
committer | Shaquille Johnson <ssjohnson@google.com> | 2024-02-28 22:14:05 +0000 |
commit | 69c92a092e781b48e0930b200dec16416e987b5d (patch) | |
tree | 0b1876afa1b0312c7e3c486163db2124972be0f1 | |
parent | 94646d7d190918036401fd4d63b44a9d357bc7d1 (diff) | |
download | security-69c92a092e781b48e0930b200dec16416e987b5d.tar.gz |
Update globals to log security levels on fail
Per go/keystore-error-logs-overhaul we are updating
the logs in keystore to be more clear. This adds
logging around the security level when the Hardware
type error is sent to the caller.
Test: atest keystore2_test
Change-Id: I7d41f02832a02976b3e1b6535ba0df5ed3863e53
-rw-r--r-- | keystore2/src/globals.rs | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/keystore2/src/globals.rs b/keystore2/src/globals.rs index eb755a6b..8535df1b 100644 --- a/keystore2/src/globals.rs +++ b/keystore2/src/globals.rs @@ -247,7 +247,11 @@ fn connect_keymint( } e => e, }) - .context(ks_err!("Trying to get Legacy wrapper."))?, + .context(ks_err!( + "Trying to get Legacy wrapper. Attempt to get keystore \ + compat service for security level {:?}", + *security_level + ))?, None, ) }; @@ -394,7 +398,7 @@ fn connect_secureclock() -> Result<Strong<dyn ISecureClock>> { } e => e, }) - .context(ks_err!("Trying to get Legacy wrapper.")) + .context(ks_err!("Failed attempt to get legacy secure clock.")) }?; Ok(secureclock) @@ -437,5 +441,5 @@ pub fn get_remotely_provisioned_component_name(security_level: &SecurityLevel) - _ => None, } .ok_or(Error::Km(ErrorCode::HARDWARE_TYPE_UNAVAILABLE)) - .context(ks_err!()) + .context(ks_err!("Failed to get rpc for sec level {:?}", *security_level)) } |