Beginner FriendlyFoundry
100 EXP
View results
Submission Details
Severity: low
Invalid

Many functions can be set to `external` instead of `public`

Summary

Many functions are set as public, however they are not called anywhere inside MondrianWallet contract. They can be set external instead.

Vulnerability Details

The following functions from MondrianWallet contract are set as public, but are not called anywhere in the code. They can be external:

  • getEntryPoint

  • getNonce

  • getDeposit

  • addDeposit

Impact

This restricts the functions access, giving more access control to the contract.

Tools Used

Manual review.

Recommendations

Set them external instead of public.

Updates

Lead Judging Commences

inallhonesty Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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