The Standard

The Standard
DeFiHardhat
20,000 USDC
View results
Submission Details
Severity: low
Invalid

The `SmartVaultManagerV5.sol` contract is not initialized in the `initialize()` function

Impact

Upgradeable contracts are set up using the initialize() function.

SmartVaultManagerV5.sol being such a contract, it should be set up in this function.

Currently, the initialization is handled by the hardhat upgrades functionnality which takes care of everyting.

Throughout the evolution of the project, the framework used may switch from hardhat to another one (e.g. foundry) which doesn't have such a functionnality.

As a result, this contract might never get initialized (or worse, initialized by a malicious user) which could lead to unexpected behaviors.

Tools used

Manual analysis

Recommended mitigation steps

Set up all the storage variables and permissions in the initialize() function

Updates

Lead Judging Commences

hrishibhat Lead Judge almost 2 years ago
Submission Judgement Published
Invalidated
Reason: Out of scope
Assigned finding tags:

informational/invalid

Support

FAQs

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

Give us feedback!