Fix signature error message (#1361)

* Fix signature error * Update CHANGELOG.md
lambdaclass · Aug 4, 2023 · 2191b5f · 2191b5f
1 parent eab16c8
commit 2191b5f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git 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
@@ -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(),
                     )))),