commit de82b56fc184e1d576f1ab50a5a38812280b7c5c
author David Drysdale <drysdale@google.com> Wed Mar 13 14:56:01 2024 +0000
committer David Drysdale <drysdale@google.com> Wed Mar 13 16:31:43 2024 +0000
tree dde2ac6bc3889920c5eb6632d5c7770f43af9f93
parent 6ed3b6ca988e581cfb30e24da8c6f7bc5f7efa3a

Drop a couple of unnecessary clones

Bug: 327526008
Test: VtsSecretkeeperTargetTest
Test: VtsAidlAuthGraphSessionTest
Change-Id: Ie5738cccad92b32bffd8229787a4db2d8494cf11

core/src/arc.rs

diff --git a/core/src/arc.rs b/core/src/arc.rs
index 8ab3974..bb7fbf0 100644
--- a/core/src/arc.rs
+++ b/core/src/arc.rs
@@ -124,7 +124,7 @@
     let arc = CoseEncrypt0Builder::new()
         .protected(protected_hdr.build())
         .unprotected(unprotected_hdr.build())
-        .try_create_ciphertext(&arc_content.payload.0.clone(), &[], |input, aad| {
+        .try_create_ciphertext(&arc_content.payload.0, &[], |input, aad| {
             cipher.encrypt(encrypting_key, input, aad, &nonce_for_enc)
         })?
         .build();

core/src/key.rs

diff --git a/core/src/key.rs b/core/src/key.rs
index 83d617c..e0169b1 100644
--- a/core/src/key.rs
+++ b/core/src/key.rs
@@ -632,7 +632,7 @@
     fn from_cbor_value(value: Value) -> Result<Self, CoseError> {
         let signature = CoseSign1::from_cbor_value(value)?;
         let payload = DiceChainEntryPayload::from_slice(
-            &signature.payload.clone().ok_or(CoseError::EncodeFailed)?,
+            signature.payload.as_ref().ok_or(CoseError::EncodeFailed)?,
         )?;
         Ok(DiceChainEntry { signature, payload })
     }