Pieces Protocol

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

Duplicate Modifier Usage on a Single Function.

Summary

The duplicateNft modified is applied twice to the divideNft function, which might result in more gas being used up.

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

Vulnerability Details

Impact

Impact: Low

Likelohood: High

Tools Used

None

Recommendations

Remove one of the onlyNftOwner modifier in the divideNft function.

Updates

Lead Judging Commences

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