Skip to content

Commit

Permalink
Remove size check in CDKGSessionManager::GetVerifiedContributions
Browse files Browse the repository at this point in the history
It's actually not true that these should always be the same. In case a
quorum is built and the total number of masternodes in the network is below
the quorum size, we might still end up having a valid quorum as long as
the total number of masternodes is >= minSize.
  • Loading branch information
codablock committed Mar 12, 2019
1 parent e21da2d commit 4ae5275
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions src/llmq/quorums_dkgsessionmgr.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -216,11 +216,6 @@ bool CDKGSessionManager::GetVerifiedContributions(Consensus::LLMQType llmqType,
{
auto members = CLLMQUtils::GetAllQuorumMembers(llmqType, quorumHash);

if (validMembers.size() != members.size()) {
// should never happen as we should always call this method with correct params
return false;
}

memberIndexesRet.clear();
vvecsRet.clear();
skContributionsRet.clear();
Expand Down

0 comments on commit 4ae5275

Please sign in to comment.