-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Description
Despite WalletAdminLib’s changeOwner and addOwner functions checking that the incoming owner does not already exist, WalletMainLib does not perform such a check in its init function. A wallet with duplicate owners could result in unexpected behavior.
Consider using require(ownerIndex[_owners[i]] == 0) inside the loop that iterates through the incoming owner array.
Metadata
Metadata
Assignees
Labels
No labels