20,000 USDC
View results
Submission Details
Severity: medium

Should add non re-entrance project while modify pool balance

Summary

protocol may lost of fund since some erc20 token have not complied with the EIP standard

Vulnerability Details

when using ERC20 token transfer may trigger hook which invoke setPool again , result in lost of ERC20 token

Impact

lost of erc20 token

Tools Used

manual

Recommendations

add openzeppelin nonReentrant() modifier

import "@openzeppelin/contracts/security/ReentrancyGuard.sol";

Support

FAQs

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

Give us feedback!