Submission Details
Severity:
Use != 0 instead of > 0 for Unsigned Integer Comparison
Summary
Changing > 0 to !=0 to save gas.
Vulnerability Details
Issue Information: G003
c4escrow/Escrow.sol::119 => if (buyerAward > 0) {
c4escrow/Escrow.sol::122 => if (i_arbiterFee > 0) {
c4escrow/Escrow.sol::126 => if (tokenBalance > 0) {
Impact
Changing > 0 to !=0 will save you about 10 gas each time.
c4escrow/Escrow.sol::119 => if (buyerAward > 0) {
c4escrow/Escrow.sol::122 => if (i_arbiterFee > 0) {
c4escrow/Escrow.sol::126 => if (tokenBalance > 0) {
Tools Used
Manual review
Recommendations
Change
c4escrow/Escrow.sol::119 => if (buyerAward > 0) {
c4escrow/Escrow.sol::122 => if (i_arbiterFee > 0) {
c4escrow/Escrow.sol::126 => if (tokenBalance > 0) {
to
c4escrow/Escrow.sol::119 => if (buyerAward != 0) {
c4escrow/Escrow.sol::122 => if (i_arbiterFee != 0) {
c4escrow/Escrow.sol::126 => if (tokenBalance != 0) {
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.