Submission Details
Severity:
Duplicate Zero Address Validation Check
The transferErcTokens function in the TokenDivider contract contains a redundant validation check for the zero address, performing the same verification twice without any additional security benefit.
Lines of Code
function transferErcTokens(address nftAddress, address to, uint256 amount) external {
// First check
if(to == address(0)) {
revert TokenDivider\_\_CantTransferToAddressZero();
}
// ... other validations ...
// Second duplicate check
if(to == address(0)) {
revert TokenDivider__CantTransferToAddressZero();
}
}
Fix Recommendation
Remove the second zero address check
This saves gas and improves contract quality
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.