aboutsummaryrefslogtreecommitdiffstats
path: root/packages
diff options
context:
space:
mode:
Diffstat (limited to 'packages')
-rw-r--r--packages/contracts/src/contracts/current/protocol/Exchange/MixinSignatureValidator.sol10
1 files changed, 4 insertions, 6 deletions
diff --git a/packages/contracts/src/contracts/current/protocol/Exchange/MixinSignatureValidator.sol b/packages/contracts/src/contracts/current/protocol/Exchange/MixinSignatureValidator.sol
index e243eade1..1e072f001 100644
--- a/packages/contracts/src/contracts/current/protocol/Exchange/MixinSignatureValidator.sol
+++ b/packages/contracts/src/contracts/current/protocol/Exchange/MixinSignatureValidator.sol
@@ -26,7 +26,7 @@ contract MixinSignatureValidator is
MSignatureValidator
{
enum SignatureType {
- Invalid,
+ Illegal, // Default value
Caller,
Ecrecover,
EIP712,
@@ -50,11 +50,9 @@ contract MixinSignatureValidator is
bytes32 r;
bytes32 s;
- // Zero is always an invalid signature
- if (signatureType == SignatureType.Invalid) {
- require(signature.length == 1);
- isValid = false;
- return;
+ // Always illegal signature
+ if (signatureType == SignatureType.Illegal) {
+ revert();
// Implicitly signed by caller
} else if (signatureType == SignatureType.Caller) {