Pieces Protocol

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

Duplicate "onlyNftOwner" modifier used

Summary:

The Duplicate "onlyNftOwner" modifier used in line 109 is repeated twice.

Vulnerability Details:

Affected code line 109

function divideNft(address nftAddress, uint256 tokenId, uint256 amount) onlyNftOwner(nftAddress, tokenId) onlyNftOwner(nftAddress ,tokenId) external {

nformational bug that might cause confusion to potential developers as to why the "onlyNftOwner" is used twice as an identifier instead of once.

Impact:

Low

  1. Can cause confusion for external developers working on the codebase

Tools Used:

Foundry

Recommendations:

Remove one of the duplicates used of the "onlyNftOwner" modifier

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.