From 84585727fe638f4f001618bea38373eb6c354d84 Mon Sep 17 00:00:00 2001 From: JesseTheRobot Date: Fri, 3 May 2024 12:41:05 +0000 Subject: [PATCH] fix: remove unsafe MultiAptosSigner public key slice --- src/signers/aptos.rs | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/signers/aptos.rs b/src/signers/aptos.rs index 4b2ac19..7f354c9 100644 --- a/src/signers/aptos.rs +++ b/src/signers/aptos.rs @@ -149,10 +149,9 @@ impl VerifierTrait for MultiAptosSigner { if sig_included { let signature = signatures.slice((i * 64)..((i + 1) * 64)); let pub_key_slc = pk.slice((i * 32)..((i + 1) * 32)); - let public_key = ed25519_dalek::VerifyingKey::from_bytes( - &pub_key_slc[0..32].try_into().unwrap(), - ) - .map_err(BundlrError::ED25519Error)?; + let public_key = + ed25519_dalek::VerifyingKey::from_bytes(&bytes_to_fixed_array(&pub_key_slc)?) + .map_err(BundlrError::ED25519Error)?; let sig = ed25519_dalek::Signature::from_bytes(&bytes_to_fixed_array(&signature)?); match public_key.verify(&message, &sig) { Ok(()) => (),