🐛 RegisterMainchainCommand.verify throws if aggregate BFT weight exce…

…eds MAX_UINT64
LiskArchive · Aug 1, 2023 · 9e6e59f · 9e6e59f
1 parent 3d369e9
commit 9e6e59f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/framework/src/modules/interoperability/sidechain/commands/register_mainchain.ts b/framework/src/modules/interoperability/sidechain/commands/register_mainchain.ts
@@ -119,14 +119,14 @@ export class RegisterMainchainCommand extends BaseInteroperabilityCommand<Sidech
 				};
 			}
 
+			totalWeight += currentValidator.bftWeight;
+
 			if (totalWeight > MAX_UINT64) {
 				return {
 					status: VerifyStatus.FAIL,
 					error: new Error('Total BFT weight exceeds maximum value.'),
 				};
 			}
-
-			totalWeight += currentValidator.bftWeight;
 		}
 
 		if (mainchainCertificateThreshold < totalWeight / BigInt(3) + BigInt(1)) {