Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Real Glossy Rhino - Anyone can Claim reward On Behalf of depositor or claimer via claimRewardOnBehalf #92

Open
sherlock-admin2 opened this issue Dec 22, 2024 · 0 comments
Labels
Won't Fix The sponsor confirmed this issue will not be fixed

Comments

@sherlock-admin2
Copy link

Real Glossy Rhino

Medium

Anyone can Claim reward On Behalf of depositor or claimer via claimRewardOnBehalf

Summary

The claimRewardOnBehalf function in the contract allows anyone to claim rewards on behalf of a deposit owner or claimer if they possess a valid _signature. However, there are no restrictions requiring the caller of the function to match the deposit.owner or deposit.claimer. This creates a significant vulnerability, as anyone with access to a valid _signature can claim rewards without being deposit.owner or deposit.claimer.

Vulnerability Detail

Impact

Anyone with access to a valid _signature can claim rewards without being the deposit owner or claimer.

Code Snippet

https://github.com/sherlock-audit/2024-11-tally/blob/main/staker/src/extensions/GovernanceStakerOnBehalf.sol#L250-L274

Tool used

Manual Review, Foundry

Recommendation

Only deposit.owner or deposit.claimer should be able to call claimRewardOnBehalf

@sherlock-admin3 sherlock-admin3 added the Won't Fix The sponsor confirmed this issue will not be fixed label Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Won't Fix The sponsor confirmed this issue will not be fixed
Projects
None yet
Development

No branches or pull requests

2 participants