Bid Beasts

First Flight #49
Beginner FriendlyFoundrySolidityNFT
100 EXP
View results
Submission Details
Impact: low
Likelihood: medium
Invalid

Typo in CurrenTokenID Variable

Root + Impact

Typographical Error in CurrenTokenID Variable Name May Cause Maintenance Issues

Description

The variable CurrenTokenID is misspelled (missing "t"). While not a runtime bug, this introduces readability issues, reduces maintainability, and increases audit/usage risks.

  • Variables should be named consistently and correctly to improve readability, maintainability, and avoid confusion.

  • The variable is named CurrenTokenID instead of CurrentTokenID.

uint256 public CurrenTokenID; // ❌ Typo

Risk

Likelihood: Medium

  • Readability errors are highly likely to cause confusion in audits and future code reviews.

  • Developers referencing the variable may misinterpret its purpose.

Impact: Low

  • Increases risk of human errors in contract upgrades or audits.

  • Reduces clarity for third-party integrators, impacting adoption and trust.

Proof of Concept

A developer searching for CurrentTokenID won’t find the variable, leading to misinterpretation of logic or introducing duplicate variables.

// Example: developer mistakenly assumes the variable doesn't exist
if (contract.CurrentTokenID) {
// This call will fail, as the correct variable is misspelled.
}

Recommended Mitigation

Rename the variable properly to improve readability and consistency.

- uint256 public CurrenTokenID;
+ uint256 public CurrentTokenID;
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!