DeFiFoundry
20,000 USDC
View results
Submission Details
Severity: medium
Invalid

User can't unstake currentEpoch by using unstakeAll function.

Summary

The unstakeAll function allows users to unstake from all the epochs at once, but it doesn't unstake the currentEpoch.
User can unstake immediately when epoch is same as currentEpoch by using unstake or unstakeVested function.
However, user can't unstake currentEpoch by using unstakeAll function.

Vulnerability Details

https://github.com/Cyfrin/2024-08-fjord/blob/main/src/FjordStaking.sol#L583

Impact

User can't unstake currentEpoch by using unstakeAll function.

Tools Used

Manual review

Recommendations

Adding the logic for unstaking currentEpoch in the unstakeAll function is recommended.

Updates

Lead Judging Commences

inallhonesty Lead Judge 10 months ago
Submission Judgement Published
Invalidated
Reason: Design choice

Support

FAQs

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