DeFiHardhatFoundry
250,000 USDC
View results
Submission Details
Severity: low
Valid

`plenty` balances are not migrated on L2

Summary

plenty balances are not migrated on L2

Vulnerability Details

When migration starts all of the beanEth, beanwsteth and bean3crv is transferred and migrated automatically. As these will be plenty tokens, this will make it impossible for users to claim their allocated plenty.

As the plenty amounts are also not migrated on L2, the user-accrued plenty will simply be lost.

function init() external {
// Pause beanstalk, preventing future sunrises.
s.paused = true;
s.pausedAt = uint128(block.timestamp);
emit Pause(block.timestamp);
// transfer the following whitelisted silo assets to the BCM:
// bean:eth
IERC20 beanEth = IERC20(C.BEAN_ETH_WELL);
uint256 beanEthBalance = beanEth.balanceOf(address(this));
beanEth.transfer(BCM, beanEthBalance);
// BEAN:WstETH
IERC20 beanwsteth = IERC20(C.BEAN_WSTETH_WELL);
uint256 beanwstethBalance = beanwsteth.balanceOf(address(this));
beanwsteth.transfer(BCM, beanwstethBalance);
// BEAN:3CRV
IERC20 bean3crv = IERC20(C.CURVE_BEAN_METAPOOL);
uint256 bean3crvBalance = bean3crv.balanceOf(address(this));
bean3crv.transfer(BCM, bean3crvBalance);
}

Impact

Loss of funds

Tools Used

Manual review

Recommendations

Migrate the plenty values on L2 and allow users to claim it

Updates

Lead Judging Commences

inallhonesty Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Design choice
Assigned finding tags:

`plenty` balances are not migrated on L2

Appeal created

deadrosesxyz Submitter
about 1 year ago
inallhonesty Lead Judge
about 1 year ago
inallhonesty Lead Judge about 1 year ago
Submission Judgement Published
Validated
Assigned finding tags:

`plenty` balances are not migrated on L2

Support

FAQs

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