Submission Details
Severity:
Double && operator costs more gas
Summary
Double && operator costs more gas than using a nested if statement
Vulnerability Details
It costs ~30 extra gas to use the double && operator instead of using a nested if statement.
Impact
Costs more gas
Tools Used
Foundry
Recommendations
Use a nested if statement instead
modifier onlyBuyerOrSeller() {
if (msg.sender != i_buyer) {
if (msg.sender != i_seller) {
revert Escrow__OnlyBuyerOrSeller();
}
}
_;
}
Prize pool breakdown
Total prize
40,000 USDC
nSLOC
186215 USDC / LOC
37,000 USDC
3,000 USDC
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.