Submission Details
Severity:
Use of duplicate modifier
Summary
There is use of same modifier twice in "divideNft" function of "TokenDivider.sol" which usage unnecessary gas.
Vulnerability Details
"onlyNftOwner" modifier is used twice in "divideNft" function of "TokenDivider.sol", which cost unnecessary gas.
function divideNft(address nftAddress, uint256 tokenId, uint256 amount) onlyNftOwner(nftAddress ,tokenId) onlyNftOwner(nftAddress ,tokenId) external { ........}
It cost 715771 gas with twice use of modifier .
It cost 714241 gas with single use of modifier.
Impact
Use of unnecessary gas.
Tools Used
Manual review.
Recommendations
Use only "onlyNftOwner" for once.
Rewards breakdown
nSLOC
181This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.