40,000 USDC
View results
Submission Details
Severity: medium

No minimum arbiter fee

Summary

The arbiter should have incentive to resolve disputes when they occur

Vulnerability Details

In Escrow.sol, the resolveDispute() function has the following,

if (i_arbiterFee > 0) {
i_tokenContract.safeTransfer(i_arbiter, i_arbiterFee);

Impact

The arbiter might not have incentive to resolve disputes making funds get locked in the contract.

Tools Used

Manual Review

Recommendations

In the constructor of Escrow.sol. you can use the following

if (arbiterFee < 0.01 ether) revert Escrow__BelowMinimumFee();

Support

FAQs

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