Summary

There is no check for the price is not zero, this must be checked.

Vulnerability Details

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

Impact

If price is zero, This could lead to transactions where no tokens are exchanged.

Tools Used

Recommendations