aboutsummaryrefslogtreecommitdiffstats
path: root/packages/contracts
diff options
context:
space:
mode:
authorRemco Bloemen <remco@wicked.ventures>2018-02-07 16:07:22 +0800
committerAmir Bandeali <abandeali1@gmail.com>2018-04-21 04:56:16 +0800
commit3133ca00815763ff2b7ef768169df2e3b6c10677 (patch)
tree48a95ff0957104de3f51b9e435ee60d604bb3a50 /packages/contracts
parent0f720223a90204b6c29b80a986f790fd549e93db (diff)
downloaddexon-sol-tools-3133ca00815763ff2b7ef768169df2e3b6c10677.tar
dexon-sol-tools-3133ca00815763ff2b7ef768169df2e3b6c10677.tar.gz
dexon-sol-tools-3133ca00815763ff2b7ef768169df2e3b6c10677.tar.bz2
dexon-sol-tools-3133ca00815763ff2b7ef768169df2e3b6c10677.tar.lz
dexon-sol-tools-3133ca00815763ff2b7ef768169df2e3b6c10677.tar.xz
dexon-sol-tools-3133ca00815763ff2b7ef768169df2e3b6c10677.tar.zst
dexon-sol-tools-3133ca00815763ff2b7ef768169df2e3b6c10677.zip
Add EIP712 signatures (implements ZEIP-17)
Diffstat (limited to 'packages/contracts')
-rw-r--r--packages/contracts/src/contracts/current/protocol/Exchange/MixinSignatureValidator.sol44
1 files changed, 40 insertions, 4 deletions
diff --git a/packages/contracts/src/contracts/current/protocol/Exchange/MixinSignatureValidator.sol b/packages/contracts/src/contracts/current/protocol/Exchange/MixinSignatureValidator.sol
index 7edb5a372..179b1b02c 100644
--- a/packages/contracts/src/contracts/current/protocol/Exchange/MixinSignatureValidator.sol
+++ b/packages/contracts/src/contracts/current/protocol/Exchange/MixinSignatureValidator.sol
@@ -29,8 +29,25 @@ contract MixinSignatureValidator is
Invalid,
Caller,
Ecrecover,
+ EIP712,
Contract
}
+
+ bytes32 public constant orderSchemaHash = keccak256(
+ "address exchangeContractAddress",
+ "address makerAddress",
+ "address takerAddress",
+ "address makerTokenAddress",
+ "address takerTokenAddress",
+ "address feeRecipientAddress",
+ "uint256 makerTokenAmount",
+ "uint256 takerTokenAmount",
+ "uint256 makerFeeAmount",
+ "uint256 takerFeeAmount",
+ "uint256 expirationTimestamp",
+ "uint256 salt"
+ );
+
function isValidSignature(
bytes32 hash,
@@ -44,6 +61,11 @@ contract MixinSignatureValidator is
require(signature.length >= 1);
SignatureType signatureType = SignatureType(uint8(signature[0]));
+ // Variables are not scoped in Solidity
+ uint8 v;
+ bytes32 r;
+ bytes32 s;
+
// Zero is always an invalid signature
if (signatureType == SignatureType.Invalid) {
require(signature.length == 1);
@@ -59,10 +81,10 @@ contract MixinSignatureValidator is
// Signed using web3.eth_sign
} else if (signatureType == SignatureType.Ecrecover) {
require(signature.length == 66);
- uint8 v = uint8(signature[1]);
- bytes32 r = get32(signature, 2);
- bytes32 s = get32(signature, 34);
- address recovered = ecrecover(
+ v = uint8(signature[1]);
+ r = get32(signature, 2);
+ s = get32(signature, 34);
+ recovered = ecrecover(
keccak256("\x19Ethereum Signed Message:\n32", hash),
v,
r,
@@ -70,6 +92,20 @@ contract MixinSignatureValidator is
);
isValid = signer == recovered;
return;
+
+ // Signature using EIP712
+ } else if (signatureType == SignatureType.EIP712) {
+ v = uint8(signature[1]);
+ r = get32(signature, 2);
+ s = get32(signature, 35);
+ address recovered = ecrecover(
+ keccak256(orderSchemaHash, orderHash),
+ v,
+ r,
+ s
+ );
+ isValid = signer == recovered;
+ return;
// Signature verified by signer contract
} else if (signatureType == SignatureType.Contract) {