Summary

In constructor of Escrow.sol missing zero check for arbiter and in function initiateDispute have this check if arbiter is zero address. Which mean that if by mistake deploy contract with arbiter zero address there no have way to dispute the escrow because lack of arbiter and there no have possability to add arbiter on later stage when escrow is already started.

Vulnerability Details

If the contract is deployed with zero address for arbiter in late stage buyer and seller can't start dispute

Impact

Buyer and seller can't start dispute

Tools Used

Manual Review

Recommendations

Make function to add arbiter with approve mechanism from buyer and seller.