#21 Use Ownable2Step instead of Ownable

Severity

Medium Risk

Relevant GitHub Links

https://github.com/Cyfrin/2023-12-the-standard/blob/91132936cb09ef9bf82f38ab1106346e2ad60f91/contracts/LiquidationPoolManager.sol#L11

Summary

Use Ownable2Step instead of 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: contracts/LiquidationPoolManager.sol

11: contract LiquidationPoolManager is Ownable {

Github: [11]

Impact

See Vulnerability Details

Tools Used

Manual Review

Recommendations

Use Ownable2Step instead of Ownable