Correct consensus logic in ContextualCheckInputs

Closes #2385.

diff --git a/src/main.cpp b/src/main.cpp
index a3de5e6d7b7d7eb437e9daac8d0f3234643093d2..6d8acde0c38ebbf8a887c4dca65e818fd042fbaa 100644 (file)
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -1687,11 +1687,12 @@ bool CheckTxInputs(const CTransaction& tx, CValidationState& state, const CCoins
 
 bool ContextualCheckInputs(const CTransaction& tx, CValidationState &state, const CCoinsViewCache &inputs, bool fScriptChecks, unsigned int flags, bool cacheStore, const Consensus::Params& consensusParams, std::vector<CScriptCheck> *pvChecks)
 {
-    if (!Consensus::CheckTxInputs(tx, state, inputs, GetSpendHeight(inputs), consensusParams))
-        return false;
-
     if (!tx.IsCoinBase())
     {
+        if (!Consensus::CheckTxInputs(tx, state, inputs, GetSpendHeight(inputs), consensusParams)) {
+            return false;
+        }
+
         if (pvChecks)
             pvChecks->reserve(tx.vin.size());