search

sherlock-audit / 2023-10-real-wagmi-judging

16 stars 14 forks source link

phenom - Use Ownable2Step rather than Ownable #166

Closed sherlock-admin2 closed 11 months ago

sherlock-admin2 commented 11 months ago

phenom

medium

Use Ownable2Step rather than Ownable

[L-5] Use Ownable2Step rather than Ownable

Impact:

Ownable2Step and Ownable2StepUpgradeable prevent the contract ownership from mistakenly being transferred to an address that cannot handle it, by requiring the recipient of the owner permissions to actively accept via a contract call of its own.

Instances (2):

File: LiqBorrowingMan/Vault.sol

8: contract Vault is Ownable, IVault {

File: LiqBorrowingMan/abstract/OwnerSettings.sol

6: abstract contract OwnerSettings is Ownable {
sherlock-admin2 commented 11 months ago

1 comment(s) were left on this issue during the judging contest.

tsvetanovv commented:

Low