stake.link

stake.link
DeFiHardhatBridge
27,500 USDC
View results
Submission Details
Severity: low
Valid

Upgradeable contract is missing a `__gap[50]` storage variable to allow for new storage variables in later versions

Summary

Upgradeable contract is missing a __gap[50] storage variable to allow for new storage variables in later versions

Vulnerability Details

See this link for a description of this storage variable. While some contracts may not currently be sub-classed adding the variable now protects against forgetting to add it in the future

Impact

Instances (3):

File: contracts/core/sdlPool/SDLPoolPrimary.sol
11: contract SDLPoolPrimary is SDLPool {

11

File: contracts/core/sdlPool/SDLPoolSecondary.sol
11: contract SDLPoolSecondary is SDLPool {

11

File: contracts/core/sdlPool/base/SDLPool.sol
15: contract SDLPool is RewardsPoolController, IERC721Upgradeable, IERC721MetadataUpgradeable {

15

Tools Used

Recommendations

Updates

Lead Judging Commences

0kage Lead Judge over 1 year ago
Submission Judgement Published
Validated
Assigned finding tags:

storage-gap

Lack of storage gaps in SDLPool might impact storage of SDLPoolPrimary and SDLPoolSecondary if new storage introduced in future.

Support

FAQs

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