Omniscia Alliance Block Audit
StakeReceiverAutoStake Static Analysis Findings
StakeReceiverAutoStake Static Analysis Findings
SRA-01S: Variable Shadowing
Type | Severity | Location |
---|---|---|
Language Specific | Minor | StakeReceiverAutoStake.sol:L14 |
Description:
The delegateStake
function's stake
argument shadows the stake
function implementation of AutoStake
.
Example:
contracts/autostake-features/StakeReceiverAutoStake.sol
14function delegateStake(address staker, uint256 stake) virtual override public {15 require(stake > 0, "delegateStake::No stake sent");16 require(staker != address(0x0), "delegateStake::Invalid staker");17 _stake(stake, staker, false);18}
Recommendation:
We advise that the shadowing collission is alleviated by renaming the stake
argument to _stake
.
Alleviation:
The variable shadowing was left as is in the codebase as it is an instance of a function versus variable shadowing with inexistent impact.