15,000 USDC
View results
Submission Details
Severity: gas
Valid

`++i`/`i++` should be `unchecked{++i}`/`unchecked{i++}` when it is not possible for them to overflow, as is the case when used in `for`- and `while`-loops

Summary

++i/i++ should be unchecked{++i}/unchecked{i++} when it is not possible for them to overflow, as is the case when used in for- and while-loops

Vulnerability Details

The unchecked keyword is new in solidity version 0.8.0,
so this only applies to that version or higher, which
these instances are.
This saves 30-40 gas per loop

Instances (2):

File: src/DSCEngine.sol
119: for (uint256 i = 0; i < tokenAddresses.length; i++) {
355: for (uint256 i = 0; i < s_collateralTokens.length; i++) {

Link to code - https://github.com/Cyfrin/2023-07-foundry-defi-stablecoin/tree/main/src/DSCEngine.sol

Tools Used

Manual inspection

Recommendations

Consider using unchecked{++i}/unchecked{i++}

Support

FAQs

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