Merge bitcoin#7500: Correctly report high-S violations

9d95187 Correctly report high-S violations (Pieter Wuille)
knocte · Feb 10, 2016 · 2f3f4af · 2f3f4af
2 parents b93f078 + 9d95187
commit 2f3f4af
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/script/interpreter.cpp b/src/script/interpreter.cpp
@@ -165,7 +165,10 @@ bool static IsLowDERSignature(const valtype &vchSig, ScriptError* serror) {
         return set_error(serror, SCRIPT_ERR_SIG_DER);
     }
     std::vector<unsigned char> vchSigCopy(vchSig.begin(), vchSig.begin() + vchSig.size() - 1);
-    return CPubKey::CheckLowS(vchSigCopy);
+    if (!CPubKey::CheckLowS(vchSigCopy)) {
+        return set_error(serror, SCRIPT_ERR_SIG_HIGH_S);
+    }
+    return true;
 }
 
 bool static IsDefinedHashtypeSignature(const valtype &vchSig) {