From 4e955c58e13cfe089208f6b23b195d395ad99baa Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jorge=20Tim=C3=B3n?= <jtimon@jtimon.cc>
Date: Tue, 4 Jul 2017 17:45:46 +0200
Subject: [PATCH] Near-Bugfix: Reestablish consensus check removed in 8d7849b

in 8d7849b6db5f54dc32fe4f8c6c7283068473cd21

This can potentially prevent an overflow that could at least in theory
allow the creation of money.
---
 src/validation.cpp | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/src/validation.cpp b/src/validation.cpp
index 8899bd2cdc..2e2a89bcf4 100644
--- a/src/validation.cpp
+++ b/src/validation.cpp
@@ -1638,6 +1638,10 @@ static bool ConnectBlock(const CBlock& block, CValidationState& state, CBlockInd
                 return error("%s: Consensus::CheckTxInputs: %s, %s", __func__, tx.GetHash().ToString(), FormatStateMessage(state));
             }
             nFees += txfee;
+            if (!MoneyRange(nFees)) {
+                return state.DoS(100, error("%s: accumulated fee in the block out of range.", __func__),
+                                 REJECT_INVALID, "bad-txns-accumulated-fee-outofrange");
+            }
 
             // Check that transaction is BIP68 final
             // BIP68 lock checks (as opposed to nLockTime checks) must