Company Simulator

First Flight #51
Beginner FriendlyDeFi
100 EXP
View results
Submission Details
Impact: high
Likelihood: high
Invalid

Refunding handling does not check for transfer result

Refunding handling does not check for result

Description

The send(msg.sender, value) result is ignored, however, the call can fail.

Risk

Likelihood: High

Impact: High

Mitigation

Use the pull pattern or proper error handling when sending ETH

Updates

Lead Judging Commences

0xshaedyw Lead Judge about 1 month ago
Submission Judgement Published
Invalidated
Reason: Lack of quality
Assigned finding tags:

Invalid

Support

FAQs

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

Give us feedback!