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 2 years ago
Submission Judgement Published
Invalidated
Reason: Known issue

Support

FAQs

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

Give us feedback!