stake.link

stake.link
DeFiHardhatBridge
27,500 USDC
View results
Submission Details
Severity: low
Invalid

There is no need to initialize variables with 0

Summary

Since the variables are automatically set to 0 when created, it is redundant to initialize it with 0 again.

Vulnerability details

RewardsInitiator.sol ( #L54, #L66, #L89 ):

for (uint256 i = 0; i < strategies.length; ++i) {
for (uint256 i = 0; i < strategiesToUpdate.length; ++i) {
for (uint256 i = 0; i < strategiesToUpdate.length; ++i) {

SDLPool.sol ( #L163 ):

for (uint256 i = 0; i < _lockIds.length; ++i) {

SDLPoolCCIPController.sol ( #L120 ):

for (uint256 i = 0; i < _tokens.length; ++i) {

SDLPoolCCIPControllerPrimary.sol ( #L64, #L68, #L90, #L174, #L192, #L253, #L264, #L354 ):

for (uint256 i = 0; i < numDestinations; ++i) {
for (uint256 i = 0; i < tokens.length; ++i) {
ffor (uint256 i = 0; i < numDestinations; ++i) {
for (uint256 i = 0; i < whitelistedChains.length; ++i) {
for (uint256 i = 0; i < _tokens.length; i++) {
for (uint256 i = 0; i < _rewardTokens.length; ++i) {
for (uint256 i = 0; i < _rewardTokens.length; ++i) {
for (uint256 i = 0; i < _tokenAmounts.length; ++i) {

SDLPoolCCIPControllerSecondary.sol ( #L152, ):

for (uint256 i = 0; i < numRewardTokens; ++i) {

SDLPoolSecondary.sol ( #L91, #L108, #L387, #L454, ):

for (uint256 i = 0; i < numNewLocks; ++i) {
for (uint256 i = 0; i < _lockIds.length; ++i) {
uint256 i = 0;
for (uint256 i = 0; i < _lockIds.length; ++i) {

WrappedTokenBridge.sol ( #L143 ):

for (uint256 i = 0; i < _tokens.length; ++i) {

Tools Used

Manual Analysis

Updates

Lead Judging Commences

0kage Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.