DeFiFoundrySolidity
16,653 OP
View results
Submission Details
Severity: low
Invalid

Spelling Error in Variable Name Could Impact Code Maintainability

Summary

The variable totalAvailabe is misspelled in _freeFunds function (should be totalAvailable). While this doesn't impact functionality as variable names don't affect bytecode, it reduces code quality and could lead to confusion during maintenance.

Vulnerability Details

The typo occurs in the variable declaration: uint256 totalAvailabe where "available" is missing an 'l'.

Impact

Low/Informational impact:

  1. Code readability is slightly reduced

  2. Search/replace operations during maintenance could miss this variable

  3. Future developers might accidentally use correct spelling, creating two variables

Recommendation

function _freeFunds(uint256 _amount) internal override {
uint256 totalAvailable = transmuter.getUnexchangedBalance(address(this)); // Fixed spelling
if (_amount > totalAvailable) {
transmuter.withdraw(totalAvailable, address(this));
} else {
transmuter.withdraw(_amount, address(this));
}
}
Updates

Appeal created

inallhonesty Lead Judge 6 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
inallhonesty Lead Judge 6 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.