We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CoinbaseSmartWallet::executeBatch
MultiOwnable::_addOwner
_addOwnerAtIndex
nextOwnerIndex
MutliOwnable::_initializeOwners
_getMultiOwnableStorage()
MultiOwnable::_initializeOwners
LGTM
The text was updated successfully, but these errors were encountered:
McToady issue #2
e9a0bd9
3docSec marked the issue as satisfactory
Sorry, something went wrong.
No branches or pull requests
Lines of code
Vulnerability details
Comments
CoinbaseSmartWallet::executeBatch
.MultiOwnable::_addOwner
with_addOwnerAtIndex
nextOwnerIndex
inMutliOwnable::_initializeOwners
rather than read from storage in each loop iteration._getMultiOwnableStorage()
inMultiOwnable::_initializeOwners
to avoid having to call it twice.Conclusion
LGTM
The text was updated successfully, but these errors were encountered: