20,000 USDC
View results
Submission Details
Severity: low

Functions calling contracts/addresses with transfer hooks are missing reentrancy guards

Summary

The smart contract is missing reentrancy guards when calling contracts/addresses with transfer hooks, potentially leading to read-only reentrancy attacks.

Vulnerability Details

The functions in the smart contract that interact with contracts or addresses containing transfer hooks lack reentrancy guards. This omission could expose users of the protocol to read-only reentrancy attacks, creating a security risk.

Code snippet

File: Fees.sol
43: IERC20(WETH).transfer(staking, IERC20(WETH).balanceOf(address(this)));

Impact

The absence of reentrancy guards in functions with transfer hooks could allow attackers to exploit the contract's vulnerability. They may manipulate the flow of the contract, potentially leading to unauthorized read access or undesired behavior, jeopardizing user funds and data.

Tools Used

Manual Review

Recommendations

Implement reentrancy guards

Support

FAQs

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