Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Severity: low
Valid

Missing Storage Gap

Summary

The MondrianWallet2 contract does not include a storage gap, which is important for adding new state variables in future upgrades without causing storage issues.

Vulnerability Details

Upgradeable contracts should have a storage gap to prevent storage layout conflicts when new variables are added. The current contract does not have this gap, which could lead to problems in future upgrades.

Impact

  • Storage Collisions: New state variables in future upgrades could overwrite existing storage slots.

  • Upgrade Issues: Makes it harder to upgrade the contract safely.

Tools Used

Manual review

Recommendations

Add a storage gap to the MondrianWallet2 contract

+ uint256[50] private __gap;
Updates

Lead Judging Commences

bube Lead Judge about 1 year ago
Submission Judgement Published
Validated
Assigned finding tags:

Missing storage gap

Support

FAQs

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