This function can be changed from public to private visibility.
Since there is no need to receive an external call from other contracts or users. It is safer for only the contract to have access.
Low impact, the computeEscrowAddress function does not need to be called by other contracts or users, it might be useful to change it to private to increase the security of the contract.
Manual code review
I suggest changing the visibility from public to private in the computeEscrowAddress()
function of the EscrowFactory.sol
contract.
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.