Severity: Low
Confidence: High
Solidity has two error handling functions: require() and assert(). require() is used for validating user input and other conditions that must be met for the code to execute successfully. When a require() statement fails, the transaction is reverted and gas is consumed. assert() is used for checking internal errors in the code, and when it fails, all gas is consumed and the transaction is reverted. However, assert() is discouraged as it can cause serious problems, such as wasted gas and denial of service attacks. Therefore, it's recommended to use require() instead of assert() in most cases.
use require instead
https://github.com/Cyfrin/2023-09-ditto/blob/main/contracts/bridges/BridgeReth.sol#L106
use require instead
https://github.com/Cyfrin/2023-09-ditto/blob/main/contracts/facets/MarginCallSecondaryFacet.sol#L87
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.