Summary
The developers mistyped boostState.minBoost = 1e4, instead they wrote boostState.minBoost = 1e18
Vulnerability Details
constructor(
address \_rewardToken,
address \_stakingToken,
address \_controller,
uint256 \_maxEmission,
uint256 \_periodDuration
) {
rewardToken = IERC20(\_rewardToken);
stakingToken = IERC20(\_stakingToken);
controller = \_controller;
_grantRole(DEFAULT_ADMIN_ROLE, msg.sender);
_grantRole(CONTROLLER_ROLE, _controller);
boostState.maxBoost = 25000;
boostState.minBoost = 1e18;
boostState.boostWindow = 7 days;
uint256 currentTime = block.timestamp;
uint256 nextPeriod = ((currentTime / _periodDuration) * _periodDuration) + _periodDuration;
periodState.periodStartTime = nextPeriod;
periodState.emission = _maxEmission;
TimeWeightedAverage.createPeriod(
periodState.votingPeriod,
nextPeriod,
_periodDuration,
0,
10000
);
}
This would lead to reverting in the calculateBoost becuase of this line:
BoostCalculator.BoostParameters memory params = BoostCalculator.BoostParameters({
maxBoost: boostState.maxBoost,
minBoost: boostState.minBoost,
boostWindow: boostState.boostWindow,
totalWeight: boostState.totalWeight,
totalVotingPower: boostState.totalVotingPower,
votingPower: boostState.votingPower
});
uint256 boost = BoostCalculator.calculateBoost(veBalance, totalVeSupply, params);
function calculateBoost(uint256 veBalance, uint256 totalVeSupply, BoostParameters memory params)
internal
pure
returns (uint256)
{
uint256 boostRange = params.maxBoost - params.minBoost;
}
Impact
you can never use applyBoost function
Tools Used
Recommendations
BoostState.minBoost has to be equal to 1e4