The Standard

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

Wrong contract version

Summary

Wrong version variable will lead to confusion.

Vulnerability Details

In the Status function a constant variable called version is used:

uint8 private constant version = 2;
function status() external view returns (Status memory) {
return Status(address(this), minted, maxMintable(), euroCollateral(),
getAssets(), liquidated, version, vaultType);
}

Impact

This may lead to confusion when a user retrieves the data as he will think he is interacting with SmartVaultV2, but in fact it will be SmartVaultV3.

Tools Used

Manual review

Recommendations

Change the constant version variable as follows:
uint8 private constant version = 3;

Updates

Lead Judging Commences

hrishibhat Lead Judge almost 2 years ago
Submission Judgement Published
Validated
Assigned finding tags:

vault-version

Support

FAQs

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

Give us feedback!