Open hats-bug-reporter[bot] opened 2 months ago
For now, these submissions will be judged as well.
@0xmahdirostami Although the functions have the same implementation, they do not break anything. This is done to distinguish between external/internal bribes but with the same functionality
Just the name fees for internal_bribes, and bribes for external_bribes, but nothing breaks, etc.
Lead: Clarification is needed for this issue. If the intention is to duplicate the same implementation and use claimFees instead of claimBribes to distinguish between external and internal bribes, then it is informational and invalid in this context. However, if there are specific vaults for fees and it functions similarly to Velodrome, then the issue is of low severity.
Github username: @burhankhaja Twitter username: imaybeghost Submission hash (on-chain): 0x1ea56a2986b80c852884053c4915b7f8158a1ca74202c7d65a2695d9691f23af Severity: low
Description: claimFees is supposed to give out claiming fee rewards
while the claimBribes is supposed to give out bribing rewards
But both the functions are exactly the same, only the names are different, since the bribe contract was out of scope, i didn't went too deep into the implementation. but it obvious that either one of the two functions is breaking.
Recommendation\ Consider either removing one of the functions, or double checking its implementation in bribes contract and re-implement accordingly.