Now using hex for asm memory addresses. (#1429)

2733cb4a · Nicolás Venturo · GitHub · 03765e08 · 2733cb4a
Unverified Commit 2733cb4a authored Oct 17, 2018 by Nicolás Venturo Committed by GitHub Oct 17, 2018
Show whitespace changes
Inline Side-by-side

Showing with 3 additions and 3 deletions

ECDSA.sol contracts/cryptography/ECDSA.sol +3 -3

No files found.
--- 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