divideNft
function, the modifier onlyNftOwner(nftAddress, tokenId)
is applied twice on the same function. This redundancy does not provide any additional security or functionality.
`TokenDivider::divideNft` function.
onlyNftOwner(nftAddress, tokenId)
modifier twice can lead to:Unnecessary Gas Consumption: Since the modifier is evaluated twice, it unnecessarily consumes more gas, even though the second evaluation doesn't provide any additional functionality.
onlyNftOwner(nftAddress, tokenId)
modifier just once** **to ensure efficiency and maintain readability in the code.The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.