Part 2
Zaros
PerpetualsDEXFoundrySolidity
70,000 USDC
View results
Previous Next
Submission Details
Severity: low
Invalid

In FeeDistributionBranch.sol::claimFees() there is no reentrancy guard.

shrxyeh

Summary

The function claimFees(...) lacks reentrancy protection, posing a risk if future features are introduced.

Vulnerability Details

  • The function performs token transfers without reentrancy guards.

  • Although current logic sets the user's fee balance to zero, future changes could introduce risks.

IERC20(weth).safeTransfer(msg.sender, amountToClaim);

Impact

Potential reentrancy attacks if new features or external calls are added.

Tools Used

Manual code review.

Recommendations

  • Add reentrancy guards to critical functions.

  • Review future changes for reentrancy risks.

Updates

Lead Judging Commences

inallhonesty Lead Judge 10 months ago
Submission Judgement Published
Invalidated
Reason: Lack of quality

Support

FAQs

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

Give us feedback!