summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorShaquille Johnson <ssjohnson@google.com>2024-02-28 22:14:05 +0000
committerShaquille Johnson <ssjohnson@google.com>2024-02-28 22:14:05 +0000
commit69c92a092e781b48e0930b200dec16416e987b5d (patch)
tree0b1876afa1b0312c7e3c486163db2124972be0f1
parent94646d7d190918036401fd4d63b44a9d357bc7d1 (diff)
downloadsecurity-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.rs10
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))
}