summaryrefslogtreecommitdiff
path: root/security/keymint/support/remote_prov_utils.cpp
diff options
context:
space:
mode:
authorDavid Drysdale <drysdale@google.com>2022-03-15 07:35:02 +0000
committerGerrit Code Review <noreply-gerritcodereview@google.com>2022-03-15 07:35:02 +0000
commitceedbb0d1af2062a46199a8cb8177a6fa6d53880 (patch)
tree0faf3b389d0658dbe02c8000f49e142f885b8c57 /security/keymint/support/remote_prov_utils.cpp
parentf325a1756c83c16fcd388976c10cd3b6300ec59e (diff)
parent08696a75435a4eb5ea826bf92ecd2a59e9f07e7a (diff)
Merge "Prevent test crashes if device sets invalid curve"
Diffstat (limited to 'security/keymint/support/remote_prov_utils.cpp')
-rw-r--r--security/keymint/support/remote_prov_utils.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/security/keymint/support/remote_prov_utils.cpp b/security/keymint/support/remote_prov_utils.cpp
index a365a3b92f..0dbea5b68c 100644
--- a/security/keymint/support/remote_prov_utils.cpp
+++ b/security/keymint/support/remote_prov_utils.cpp
@@ -225,7 +225,7 @@ ErrMsgOr<EekChain> generateEekChain(int32_t supportedEekCurve, size_t length,
bytevec prev_priv_key;
for (size_t i = 0; i < length - 1; ++i) {
auto keyPair = generateKeyPair(supportedEekCurve, false);
- if (!keyPair) keyPair.moveMessage();
+ if (!keyPair) return keyPair.moveMessage();
auto [pub_key, priv_key] = keyPair.moveValue();
// The first signing key is self-signed.
@@ -242,7 +242,7 @@ ErrMsgOr<EekChain> generateEekChain(int32_t supportedEekCurve, size_t length,
prev_priv_key = priv_key;
}
auto keyPair = generateKeyPair(supportedEekCurve, true);
- if (!keyPair) keyPair.moveMessage();
+ if (!keyPair) return keyPair.moveMessage();
auto [pub_key, priv_key] = keyPair.moveValue();
auto coseKey = constructCoseKey(supportedEekCurve, eekId, pub_key);