WalletMainLib’s init function verifies that all confirmation requirements are larger than zero, and that none of such requirements are larger than the total number of owners. However, there is no check to ensure that requiredMinor <= requiredMajor <= requiredAdmin. Consider adding this check in WalletMainLib’s init function, as well as in the WalletAdminLib functions that change requirements: changeRequiredAdmin, changeRequiredMajor and changeRequiredMinor.
WalletMainLib’s init function verifies that all confirmation requirements are larger than zero, and that none of such requirements are larger than the total number of owners. However, there is no check to ensure that requiredMinor <= requiredMajor <= requiredAdmin. Consider adding this check in WalletMainLib’s init function, as well as in the WalletAdminLib functions that change requirements: changeRequiredAdmin, changeRequiredMajor and changeRequiredMinor.