QA Report #584
Labels
bug
Something isn't working
old-submission-method
QA (Quality Assurance)
Assets are not at risk. State handling, function incorrect as to spec, issues with clarity, syntax
Summary
We list 1 low-critical finding:
(Low) It’s better to define uint8 founderPct
Impact
founderPct
is defined as uint256, but it’s used for both uint8 and uint256.Proof of Concept
founderPct is uint256 but is used by uin8:
https://github.com/code-423n4/2022-09-nouns-builder/blob/main/src/token/Token.sol#L82
https://github.com/code-423n4/2022-09-nouns-builder/blob/main/src/token/Token.sol#L88
But L102 uint256 again:
https://github.com/code-423n4/2022-09-nouns-builder/blob/main/src/token/Token.sol#L102
Recommended Mitigation Steps
Define uint8 rather than uint256 in L82.
The text was updated successfully, but these errors were encountered: