The _nextTokenId variable in the UpliftOnlyExample contract appears to be unused throughout the project.
As the name suggests, the _nextTokenId variable is intended to store the ID of the next NFT to be minted in the UpliftOnlyExample contract. However, it is neither assigned a value nor utilized to store the intended data.
As a result, the contract will lose track of the NFT count, fail to monitor the minted NFTs, or potentially mint NFTs with incorrect token IDs in the UpliftOnlyExample contract.
Manual Review
To ensure the proper functioning of the contract and the project, the _nextTokenId variable should be utilized in the appropriate locations to accurately track and manage the minting process.
If redundant remove them from the contract to save gas and misinformations.
Please read the CodeHawks documentation to know which submissions are valid. If you disagree, provide a coded PoC and explain the real likelyhood and the detailed impact on the mainnet without any supposition (if, it could, etc) to prove your point.
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.