Pieces Protocol

First Flight #32
Beginner FriendlyFoundrySolidityNFT
100 EXP
View results
Submission Details
Severity: medium
Invalid

The visibility of getter functions should be adjusted

Summary

The visibility getBalanceOf, getErc20TotalMintedAmount, getErc20InfoFromNft, and getOrderPrice is set to `public`

Vulnerability Details

The getter functions' visibility should be adjusted if they are not intended to be called internally

Impact

It needs to be adjusted to prevent unauthorized access

Tools Used

Github Copilot

Recommendations

The getter functions mentioned above should be marked as `external`

Updates

Lead Judging Commences

fishy Lead Judge 8 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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