Closed code423n4 closed 2 years ago
jonah1005
divider is set to 1e18. We can save a lot of SLOAD if we use a constant instead.
divider
SLOAD
https://github.com/code-423n4/2021-10-covalent/blob/main/contracts/DelegatedStaking.sol#L9
None
Please refer to the solidity doc. https://docs.soliditylang.org/en/latest/contracts.html#constant-and-immutable-state-variables
duplicate of #27
Duplicate of #27
Handle
jonah1005
Vulnerability details
Impact
divider
is set to 1e18. We can save a lot ofSLOAD
if we use a constant instead.Proof of Concept
https://github.com/code-423n4/2021-10-covalent/blob/main/contracts/DelegatedStaking.sol#L9
Tools Used
None
Recommended Mitigation Steps
Please refer to the solidity doc. https://docs.soliditylang.org/en/latest/contracts.html#constant-and-immutable-state-variables