40,000 USDC
View results
Submission Details
Severity: low

[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.

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.