# Wrong comment in Standard Governor's execute function Low/Info · Three Sigma · Stablecoin framework · 8th January, 2024 Finding 3S-M^0-N08 of the M^0 Minter Gateway security review. - Protocol: https://www.m0.org/ - Report: /reports/m-0 - Codebase: https://github.com/0xsimao/ttg/tree/a8127901fa1f24a2e821cf4d9854a1aa6ac8088c - Source: https://cdn.sanity.io/files/qoqld077/production/1cdafafad874aba76e062ad8c216c98338c096db.pdf --- ### Description The following comment is present in [StandardGovernor](https://github.com/MZero-Labs/ttg/blob/a8127901fa1f24a2e821cf4d9854a1aa6ac8088c/src/StandardGovernor.sol)'s execute function: // Proposals have voteStart=N and voteEnd=N, and can be executed only during epochs N+1 and N+2. This, however, is not the current behaviour of the contract. As confirmed by the client, approved proposals can only be executed 1 epoch after voteEnd and not 2 epochs as the comment suggests. ### Recommendation Update the comment so it explains the actual behaviour of epoch execution: // Proposals have voteStart=N and voteEnd=N, and can be executed only during N+1 epoch. ### Status Addressed in [#22bd53c](https://github.com/MZero-Labs/ttg/commit/22bd53c50b4224217a4548d85c9ac8615f49f31f). --- Related findings: - [CEI pattern is not followed in `GovernorTimelock::executeProposals()`](https://0xsimao.com/findings/maple-finance-ii-cei-followed-timelock-proposals): Maple Governor Timelock - [Wrong transformation in function previewMintDebt(...)](https://0xsimao.com/findings/fuji-finance-transformation-preview-mint-debt): Fuji Finance