update ecdsa patch fixes

succinctlabs · Dec 5, 2024 · 459ac0f · 459ac0f
1 parent 43792f1
commit 459ac0f
Show file tree

Hide file tree

Showing 5 changed files with 398 additions and 362 deletions.
diff --git a/crates/core/executor/src/hook.rs b/crates/core/executor/src/hook.rs
@@ -6,7 +6,7 @@ use hashbrown::HashMap;
 use sp1_curves::{
     ecdsa::RecoveryId as ecdsaRecoveryId,
     k256::{Invert, RecoveryId, Signature, VerifyingKey},
-    p256::{Signature as p256Signature, VerifyingKey as p256VerifyingKey},
+    p256::{Invert as p256Invert, Signature as p256Signature, VerifyingKey as p256VerifyingKey},
 };
 
 use crate::Executor;
@@ -223,7 +223,8 @@ pub fn hook_r1_ecrecover(_: HookEnv, buf: &[u8]) -> Vec<Vec<u8>> {
         return vec![vec![0]];
     };
 
-    let bytes = recovered_key.to_sec1_bytes();
+    let recovered_key_encoded = recovered_key.to_encoded_point(true);
+    let bytes = recovered_key_encoded.as_bytes();
 
     let (_, s) = sig.split_scalars();
     let s_inverse = s.invert();