Submission Details
Severity:
[L-01] Change visibility of computeEscrowAddress()
Summary
This function can be changed from public to private visibility.
Vulnerability Details
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.
Impact
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.
Tools Used
Manual code review
Recommendations
I suggest changing the visibility from public to private in the computeEscrowAddress() function of the EscrowFactory.sol contract.
Prize pool breakdown
Total prize
40,000 USDC
nSLOC
186215 USDC / LOC
37,000 USDC
3,000 USDC
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.