fix: remove unsafe MultiAptosSigner public key slice

Irys-xyz · May 3, 2024 · 8458572 · 8458572
1 parent 79c6cdc
commit 8458572
Showing 1 changed file with 3 additions and 4 deletions.
diff --git 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(()) => (),