MorpheusAI

MorpheusAI
Foundry
22,500 USDC
View results
Submission Details
Severity: low
Invalid

Consider Using `Ownable2Step` rather than `Ownable`

Summary

To enhance the security and prevent inadvertent ownership transfers, it's advisable to use Ownable2Step or Ownable2StepUpgradeable.
Contracts necessitate an active confirmation from the recipient before the ownership transfer is finalized.
This mechanism serves as a safeguard against scenarios where, for instance, a typo in the address could lead to unintentional ownership changes.

By implementing a two-step confirmation process, contracts can better ensure the accurate and intentional transfer of ownership.

Vulnerability Details

File: contracts/Distribution.sol
15: contract Distribution is IDistribution, OwnableUpgradeable, UUPSUpgradeable {
File: contracts/L1Sender.sol
16: contract L1Sender is IL1Sender, ERC165, OwnableUpgradeable, UUPSUpgradeable {
File: contracts/L2MessageReceiver.sol
10: contract L2MessageReceiver is IL2MessageReceiver, OwnableUpgradeable, UUPSUpgradeable {
File: contracts/L2TokenReceiver.sol
13: contract L2TokenReceiver is IL2TokenReceiver, OwnableUpgradeable, UUPSUpgradeable {
File: contracts/MOR.sol
10: contract MOR is IMOR, ERC20Capped, ERC20Burnable, Ownable {
Updates

Lead Judging Commences

inallhonesty Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Known issue

Support

FAQs

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