DittoETH

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

Missing contract-existence checks before low-level calls

Summary

Low-level calls return success even if there is no code present at the specified address.

Vulnerability Details

There is 1 instance of this issue.

File: contracts/bridges/BridgeReth.sol
105: (bool sent,) = to.call{value: netBalance}("");
File Link Instance Count Instance Link
BridgeReth.sol 1 105

Impact

Potential to send funds to an address that cannot handle receiving them.

Tools Used

baudit: a custom static code analysis tool; manual review

Recommendations

In addition to the zero-address checks, consider adding a check to verify that <address>.code.length > 0.

Updates

Lead Judging Commences

0xnevi Lead Judge
almost 2 years ago
0xnevi Lead Judge almost 2 years ago
Submission Judgement Published
Invalidated
Reason: User input validation

Support

FAQs

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