From 56806bb295b409145547c8bd5e48f2b2b137de09 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nicol=C3=A1s=20Venturo?= Date: Wed, 17 Oct 2018 16:58:36 -0300 Subject: [PATCH] Now using hex for asm memory addresses. (#1429) (cherry picked from commit 2733cb4a411cf34e1a027ec5799298aa39ff549a) --- contracts/cryptography/ECDSA.sol | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/contracts/cryptography/ECDSA.sol b/contracts/cryptography/ECDSA.sol index 3f0ad1bd2..71c0e41ed 100644 --- a/contracts/cryptography/ECDSA.sol +++ b/contracts/cryptography/ECDSA.sol @@ -33,9 +33,9 @@ library ECDSA { // currently is to use assembly. // solium-disable-next-line security/no-inline-assembly assembly { - r := mload(add(signature, 32)) - s := mload(add(signature, 64)) - v := byte(0, mload(add(signature, 96))) + r := mload(add(signature, 0x20)) + s := mload(add(signature, 0x40)) + v := byte(0, mload(add(signature, 0x60))) } // Version of signature should be 27 or 28, but 0 and 1 are also possible versions