About Preparams Selection Logic During Reshare Phase

[treeforest]

Hello,

I noticed that during the reshare phase, new nodes always use the preparams at index 1. However, after the first reshare, in the second reshare, they might end up using the same preparams again.

Currently, the code snippet is:

preParams := p.ecdsaPreParams[0]
if isNewPeer {
	preParams = p.ecdsaPreParams[1]
	participantPeerIDs = newPeerIDs
} else {
	participantPeerIDs = oldKeyInfo.ParticipantPeerIDs
}

Would it be better to select preparams based on version % 2 and (version + 1) % 2 instead, to alternate the preparams and avoid reuse?

Looking forward to your thoughts. Thanks!

[anhthii]

@vietddude can help

[anhthii]

Thank you for reporting @treeforest , fixed in #139