diff options
author | David Drysdale <drysdale@google.com> | 2022-03-15 07:35:02 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2022-03-15 07:35:02 +0000 |
commit | ceedbb0d1af2062a46199a8cb8177a6fa6d53880 (patch) | |
tree | 0faf3b389d0658dbe02c8000f49e142f885b8c57 /security/keymint/support/remote_prov_utils.cpp | |
parent | f325a1756c83c16fcd388976c10cd3b6300ec59e (diff) | |
parent | 08696a75435a4eb5ea826bf92ecd2a59e9f07e7a (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.cpp | 4 |
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); |