Upgraded Q -> 2 from #1011 [1716984547522] #1321
Labels
2 (Med Risk)
Assets not at direct risk, but function/availability of the protocol could be impacted or leak value
duplicate-829
satisfactory
satisfies C4 submission criteria; eligible for awards
Judge has assessed an item in Issue #1011 as 2 risk. The relevant finding follows:
[L-06] Deployment script deploys bounded kerosine without calling setUnboundedKerosineVault
Issue Description:
Deployment of the BoundedKerosineVault happens without calling setUnboundedKerosineVault, that will eventually make the price of the asset 0, when admin license it and users begin to use it. The consequences will be that it won’t support depositor’s collateral rate and will limit their minting capabilities.
DeployV2.sol#L78-L82
BoundedKerosineVault boundedKerosineVault = new BoundedKerosineVault(
vaultManager,
Kerosine(MAINNET_KEROSENE),
kerosineManager
);
Recommendation:
Despite that it is not going to be used for now, call the setUnboundedKerosineVault in the script as well.
The text was updated successfully, but these errors were encountered: