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

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

File: src/ProxyFactory.sol
26: import {Ownable} from "openzeppelin/access/Ownable.sol";
37: contract ProxyFactory is Ownable, EIP712 {
81: constructor(address[] memory _whitelistedTokens) EIP712("ProxyFactory", "1") Ownable() {

Impact

DOS on owner functionalities

Tools Used

Manual

Recommendations

Use Ownable2Step rather than Ownable

Support

FAQs

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