Summary

safeTransfer function in L98 and 4 other instances are taking in 2 parameters instead of 3 parameters which is the standard according to the safeTransfer function on openzeppelin library

Vulnerability Details

from the openzeppelin library it can be confirmed that safeTransfer takes in 3 parameters and not 2
https://github.com/OpenZeppelin/openzeppelin-contracts/blob/master/contracts/token/ERC20/utils/SafeERC20.sol#L36

However using L98 as a sample, what can be seen is two parameters

This is also the case at L120, L123 & L127 of the Escrow.sol contract

Impact

Would affect all round functionality of fund transaction in the Escrow contract, which would be a huge problem

Tools Used

Solidity, Hardhat

Recommendations

Three parameters should be used and not two, except the developer was actually intending to use transfer() and not safeTransfer()