DittoETH

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

Empty `receive()` function

Summary

The receive() function is completely empty.

Vulnerability Details

There is 1 instance of this issue.

File: contracts/bridges/BridgeReth.sol
37: receive() external payable {}
File Link Instance Count Instance Link
BridgeReth.sol 1 37

Impact

The purpose of the code as written is ambiguous.

Tools Used

baudit: a custom static code analysis tool; manual review

Recommendations

If the intention is for Ether sent by a caller to be used for an actual purpose, the function should call another function (e.g. call depositEth() and use the token on the caller's behalf). At a minimum an event should be emitted to track that funds were sent directly to it.

Updates

Lead Judging Commences

0xnevi Lead Judge
about 2 years ago
0xnevi Lead Judge about 2 years ago
Submission Judgement Published
Invalidated
Reason: Other

Support

FAQs

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