Closed code423n4 closed 3 years ago
hrkrshnn
notHalted
depositMultiple
Since the call to deposit already has the notHalted modifier, doing the check during depositMultiple is redundant. Saves at least 100 gas.
100
https://github.com/code-423n4/2021-09-yaxis/blob/main/contracts/v3/Vault.sol#L194
Duplicate of #70
GalloDaSballo
commented
3 years ago GalloDaSballo
commented
3 years ago
Handle
hrkrshnn
Vulnerability details
Redundant
notHaltedmodifier indepositMultipleSince the call to deposit already has the
notHaltedmodifier, doing the check duringdepositMultipleis redundant. Saves at least100gas.https://github.com/code-423n4/2021-09-yaxis/blob/main/contracts/v3/Vault.sol#L194