Summary

There is no check for the arbiterFee is not zero.

Vulnerability Details

Add below code snippet in the constructor of Escrow.sol =>
if (arbiterFee == 0) revert Escrow__FeeZero();
Add below error in the IEscrow.sol =>
error Escrow__FeeZero();

Impact

If arbiter fee is zero, where arbitrator is not paid for their services.

Tools Used

Recommendations