The Standard

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

Lack of `_disableInitializers` call to prevent uninitialized contracts

Summary

Lack of _disableInitializers call to prevent uninitialized contracts

Vulnerability Details

Multiple contracts are using the Initializable module from OpenZeppelin. For this reason and in order to prevent leaving that contract uninitialized OpenZeppelin's documentation recommends adding the _disableInitializers function in the constructor to automatically lock the contracts when they are deployed. this will protect the contract that holds the logic business from beeing initialized by an attack.

File: contracts/SmartVaultManagerV5.sol
46: function initialize() initializer public {}

Github: [46]

Impact

See Vulnerability Details

Tools Used

Manual Review

Recommendations

Consider adding _disableInitializers in constructor

Updates

Lead Judging Commences

hrishibhat Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

informational/invalid

Support

FAQs

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