40,000 USDC
View results
Submission Details
Severity: medium

ArbiterFee should not be 0 in the constructor

Summary

ArbiterFee should not be 0 in the constructor.

Vulnerability Details

See the following code,

if (arbiterFee >= price) revert Escrow__FeeExceedsPrice(price, arbiterFee);

arbiterFee should be < price but it is possible for the value to be 0

Impact

if arbiterFee = 0, no arbiter will resolve any dispute because there is no reward for doing so.
No resolution on dispute will lead to fund always stuck in the contract at DISPUTE state.

Tools Used

Manual

Recommendations

Enforce arbiterFee > 0 in the constructor

Support

FAQs

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