Sparkn

CodeFox Inc.
DeFiFoundryProxy
15,000 USDC
View results
Submission Details
Severity: low
Valid

Use Ownable2Step rather than Ownable

Summary

Use Ownable2Step rather than Ownable

Vulnerability Details

ProxyFactory.sol utilises ownable.sol by OZ which is error prone as it does not check for recipient acceptance.

Impact

ownable does not prevent the contract ownership by not checking that the recipient of the owner permissions actively accept via a contract call of its own.

Tools Used

Manual code

Recommendations

Ownable2Step and Ownable2StepUpgradeable prevent the contract ownership from mistakenly being transferred to an address that cannot handle it.

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.

Give us feedback!