Sparkn

CodeFox Inc.
DeFiFoundryProxy
15,000 USDC
View results
Submission Details
Severity: low

Function without internal calls can be set external

Summary

Vulnerability Details

Several functions in the ProxyFactory contract have used public visibility. They should use external instead of public if no function in the contract calls them.

Impact

Does not follow solidity best practices.

Tools Used

Manual Review

Recommendations

Use external visibility instead of public.

Reference

https://ethereum.stackexchange.com/questions/19380/external-vs-public-best-practices

https://solodit.xyz/issues/use-external-instead-of-public-auditone-none-unicrow-markdown

Support

FAQs

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