GoGoPool contest - jadezti's results

Judge has assessed an item in Issue #862 as 2 risk. The relevant finding follows:

[L-1] Function requireNextActiveMultisig always returns the first Multisig Affected code MultisigManager.requireNextActiveMultisig is supposed to return the next enabled Multisig. However it always returns the first matched Multisig since it always searches from the beginning and returns the first matched item. Mitigation Suggestion: Add an extra storage variable to store the index of the current selected Multisig. When requireNextActiveMultisig is called, it searches from the last selected Multisig index and wraps through until to the the last selected Multisig location again.