Omniscia Olympus DAO Audit
GovernorAlpha Manual Review Findings
GovernorAlpha Manual Review Findings
GAA-01M: Improper Percentage Documented
Type | Severity | Location |
---|---|---|
Logical Fault | Informational | GovernorAlpha.sol:L14 |
Description:
The proposalThresholdPercent
is meant to showcase the percentage of voting power required to make a proposal, however, it is incorrectly documented as 1.00% when in reality it is 10% when the divisor of the getVotesFromPercentOfsOHMSupply
function is taken into account.
Example:
contracts/governance/GovernorAlpha.sol
12/// @notice The maximum setable proposal threshold percent13/// @notice change from original contract14function proposalThresholdPercent() public pure returns (uint) { return 100000; } // 1.00% of sOHM circulating supply : In ten-thosandaths 10000 = 1.00%
Recommendation:
We advise the documentation or value itself to be properly remediated depending on the canonical value of the proposal threshold.
Alleviation:
The value was corrected to the 1.00% indicated by its corresponding comments.