Skip to content
This repository was archived by the owner on Jan 12, 2025. It is now read-only.

Honour - Briber has no way to claim unclaimed bribes after bribing period ends as stated by the docs #291

Closed
sherlock-admin2 opened this issue Jul 15, 2024 · 1 comment
Labels
Duplicate A valid issue that is a duplicate of an issue with `Has Duplicates` label High A High severity issue. Reward A payout will be made for this issue

Comments

@sherlock-admin2
Copy link

sherlock-admin2 commented Jul 15, 2024

Honour

Medium

Briber has no way to claim unclaimed bribes after bribing period ends as stated by the docs

Summary

Briber has no way to claim unclaimed bribes after bribing period ends

Vulnerability Detail

It's stated in the docs
image
the briber can re-claim unclaimed bribes after the bribing period ends , But the BribeRewarder does not have a function for this

Impact

Broken functionality

Code Snippet

https://github.com/sherlock-audit/2024-06-magicsea/blob/main/magicsea-staking/src/rewarders/BribeRewarder.sol#L1

Tool used

Manual Review

Recommendation

function sweep() external onlyOwner {
    require(IVoter(_caller).getCurrentVotingPeriod() > _lastVotingPeriod + 1 );
    _safeTransferTo(_token(), msg.sender , balanceOfThis(_token()));
}

Duplicate of #172

@github-actions github-actions bot added duplicate Medium A Medium severity issue. labels Jul 21, 2024
@sherlock-admin2 sherlock-admin2 added the Duplicate A valid issue that is a duplicate of an issue with `Has Duplicates` label label Jul 22, 2024
@0xSmartContract 0xSmartContract added High A High severity issue. and removed Medium A Medium severity issue. labels Jul 29, 2024
@sherlock-admin4 sherlock-admin4 changed the title Slow Indigo Butterfly - Briber has no way to claim unclaimed bribes after bribing period ends as stated by the docs Honour - Briber has no way to claim unclaimed bribes after bribing period ends as stated by the docs Jul 29, 2024
@sherlock-admin4 sherlock-admin4 added the Reward A payout will be made for this issue label Jul 29, 2024
@WangSecurity
Copy link

Now a duplicate of #172, based on #164 (comment) and #164 (comment) comments.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Duplicate A valid issue that is a duplicate of an issue with `Has Duplicates` label High A High severity issue. Reward A payout will be made for this issue
Projects
None yet
Development

No branches or pull requests

4 participants