Merge pull request #1712 from winsvega/issues

issues: min transaction s value is 1
10 years ago · e66b81b152
1 changed files with 3 additions and 1 deletions
--- a/libdevcrypto/Common.cpp
+++ b/libdevcrypto/Common.cpp
@ -39,7 +39,9 @@ bool dev::SignatureStruct::isValid() const
 {
 	if (v > 1 ||
 		r >= h256("0xfffffffffffffffffffffffffffffffebaaedce6af48a03bbfd25e8cd0364141") ||
-		s >= h256("0xfffffffffffffffffffffffffffffffffffffffffffffffffffffffefffffc2f"))
+		s >= h256("0xfffffffffffffffffffffffffffffffebaaedce6af48a03bbfd25e8cd0364141") ||
+		s < h256("0x01") ||
+		r < h256("0x01"))
 		return false;
 	return true;
 }