DittoETH

Ditto
DeFiFoundryOracle
55,000 USDC
View results
Submission Details
Severity: medium
Invalid

Use assert to check the return value of the call

Summary

Use assert to check the return value of the call

Vulnerability Details

If the transfer fails, assert is triggered and the transaction is terminated, rolling back the entire state. Assert is more suitable for testing internally. Require should be used to check if the transfer was successful.

Impact

Tools Used

Recommendations

Suggested modification:

require(sent, "Transfer failed");
Updates

Lead Judging Commences

0xnevi Lead Judge
almost 2 years ago
0xnevi Lead Judge almost 2 years ago
Submission Judgement Published
Invalidated
Reason: Gas optimizations

Support

FAQs

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