Bid Beasts
First Flight #49
Beginner FriendlyFoundrySolidityNFT
100 EXP
View results
Previous Next
Submission Details
Impact: low
Likelihood: low
Invalid

LOW ISSUE: Typo in Variable Name

mostafapahlevani93

Description

  • The state variable CurrenTokenID has a typo and should be CurrentTokenID.

uint256 public CurrenTokenID; // Missing 't' in "Current"

Risk

Likelihood:

  • The typo exists permanently in the deployed contract

  • It affects code readability and professionalism

  • External integrations must use the incorrect spelling

Impact:

  • Reduced code quality and professionalism

  • Potential confusion for developers integrating with the contract

  • Inconsistent naming conventions

Proof of Concept

// External contracts must use the typo
function getCurrentTokenId() external view returns (uint256) {
return nft.CurrenTokenID(); // Must use incorrect spelling
}

Recommended Mitigation

- uint256 public CurrenTokenID;
+ uint256 public CurrentTokenID;
​
function mint(address to) public onlyOwner returns (uint256) {
- uint256 _tokenId = CurrenTokenID;
+ uint256 _tokenId = CurrentTokenID;
_safeMint(to, _tokenId);
emit BidBeastsMinted(to, _tokenId);
- CurrenTokenID++;
+ CurrentTokenID++;
return _tokenId;
}
Updates

Lead Judging Commences

cryptoghost Lead Judge 2 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.

Give us feedback!