From 2191b5fad41108bbec5158feb593c3f5126a93bf Mon Sep 17 00:00:00 2001 From: FabijanC Date: Fri, 4 Aug 2023 19:50:25 +0200 Subject: [PATCH] Fix signature error message (#1361) * Fix signature error * Update CHANGELOG.md --- CHANGELOG.md | 2 ++ vm/src/vm/runners/builtin_runner/signature.rs | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index ab33b72c76..c5e782212a 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -2,6 +2,8 @@ #### Upcoming Changes +* fix: correctly display invalid signature error message [#1361](https://github.com/lambdaclass/cairo-vm/pull/1361) + #### [0.8.5] - 2023-7-31 * fix: `Program` comparison depending on `hints_ranges` ordering [#1351](https://github.com/lambdaclass/cairo-rs/pull/1351) diff --git a/vm/src/vm/runners/builtin_runner/signature.rs b/vm/src/vm/runners/builtin_runner/signature.rs index 2c39084322..cfc0db9d0c 100644 --- a/vm/src/vm/runners/builtin_runner/signature.rs +++ b/vm/src/vm/runners/builtin_runner/signature.rs @@ -137,7 +137,7 @@ impl SignatureBuiltinRunner { match verify(&public_key, &message, &r, &s) { Ok(true) => Ok(vec![]), _ => Err(MemoryError::InvalidSignature(Box::new(( - signature.to_string(), + format!("({}, {})", signature.r, signature.s), pubkey.into_owned(), msg.into_owned(), )))),