Summary

Instead of array, we can use mapping.

Vulnerability Details

Instead of address[] private s_collateralTokens;

use this =>

mapping(address => bool) private s_collateralTokens;

function addCollateralToken(address token) external {
s_collateralTokens[token] = true;
}

function removeCollateralToken(address token) external {
s_collateralTokens[token] = false;
}

function isCollateralToken(address token) external view returns (bool) {
return s_collateralTokens[token];
}

Impact

Gas cost will be reduced.

Tools Used

Manual Review

Recommendations