The contract EggstravaganzaNFT.sol
maintains a redundant totalSupply
counter.
The contract tracks:
But ERC721 already tracks ownership through _owners
mapping, making this duplicate state.
Low - Gas inefficiency but no security impact.
Manual Review
Remove totalSupply unless explicitly needed
If needed for off-chain queries, consider making it immutable after minting
Use balanceOf or token enumeration if supply info is needed
Strategy to save gas and minimize transaction costs
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.