Small fix in shouldValidatorReport getter

poanetwork · Dec 29, 2020 · 5b8b4a3 · 5b8b4a3
1 parent 1c86654
commit 5b8b4a3
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/contracts/ValidatorSetAuRa.sol b/contracts/ValidatorSetAuRa.sol
@@ -565,7 +565,9 @@ contract ValidatorSetAuRa is UpgradeabilityAdmin, IValidatorSetAuRa {
         uint256 _blockNumber
     ) public view returns(bool) {
         uint256 currentBlock = _getCurrentBlockNumber();
-        if (_blockNumber > currentBlock) {
+        if (_blockNumber > currentBlock + 1) {
+            // we added +1 in the condition to let validator next to the malicious one correctly report
+            // because that validator will use the previous block state when calling this getter
             return false;
         }
         if (currentBlock > 100 && currentBlock - 100 > _blockNumber) {

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "posdao-contracts",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Smart contracts for POSDAO consensus",
   "main": "index.js",
   "scripts": {