Pieces Protocol

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

Redundant Modifier in function divideNft

Summary

Modifier onlyNftOwner has been associated twice with function divideNft in the contract TokenDivider.sol

Vulnerability Details

Codelines inside modifier onlyNftOwner are called twice because the modified is associated twice.

Impact

Redundant code. Extra Gas

Tools Used

Manual Instpection.

Recommendations

Remove extra onlyNftOwner modifier from the function divideNft in TokenDivider.sol

Updates

Lead Judging Commences

fishy Lead Judge 7 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.