code-423n4 / 2024-01-curves-findings

1 stars 0 forks source link

`` FeeSplitter :: claimFees `` claim transaction can revert if msg.sender is a contract that does not receive funds #1482

Closed c4-bot-9 closed 10 months ago

c4-bot-9 commented 10 months ago

Lines of code

https://github.com/code-423n4/2024-01-curves/blob/516aedb7b9a8d341d0d2666c23780d2bd8a9a600/contracts/FeeSplitter.sol#L80

Vulnerability details

Impact

transaction revert and cause a DOS attack

Tools Used

manual

Recommended Mitigation Steps

Use Safetransfer instead of tranfer

Assessed type

DoS

c4-pre-sort commented 10 months ago

raymondfam marked the issue as insufficient quality report

c4-pre-sort commented 10 months ago

raymondfam marked the issue as duplicate of #56

c4-judge commented 10 months ago

alcueca marked the issue as satisfactory

c4-judge commented 9 months ago

alcueca marked the issue as unsatisfactory: Invalid