Skip to content

Commit

Permalink
Merge pull request #431 from icon-project/fix/icon-cluster-connection
Browse files Browse the repository at this point in the history
fix: icon cluster connection issue fixed
  • Loading branch information
gcranju authored Feb 3, 2025
2 parents d81ad50 + 2f214f0 commit 2a117ee
Showing 1 changed file with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -266,10 +266,12 @@ public void recvMessageWithSignatures(
for (byte[] signature : signatures) {
byte[] validator = getValidator(messageHash, signature);
String hexValidator = bytesToHex(validator);
Context.require(isValidator(hexValidator), "Invalid signature provided");
if (!uniqueValidators.contains(hexValidator)) {
if (isValidator(hexValidator) && !uniqueValidators.contains(hexValidator)) {
uniqueValidators.add(hexValidator);
}
if (uniqueValidators.size() >= validatorsThreshold.get().intValue()) {
break;
}
}
Context.require(uniqueValidators.size() >= validatorsThreshold.get().intValue(), "Not enough valid signatures");
recvMessage(srcNetwork, _connSn, msg);
Expand Down

0 comments on commit 2a117ee

Please sign in to comment.