double reading of state variable inside a loop

[code423n4]

Handle

pants

Vulnerability details

MixinOperatorResolver.rebuildCache (addressCache[name]), isResolverCached (addressCache[name])

You can cache the value after the first read into a local variable to save the other SLOAD and also the "out of bounds" check.

[maximebrugel]

The issue is not clear...

So, we confirm for isResolverCached :

function isResolverCached() external view returns (bool) {
    bytes32[] memory requiredAddresses = resolverAddressesRequired();
    for (uint256 i = 0; i < requiredAddresses.length; i++) {
        bytes32 name = requiredAddresses[i];
        address cache_tmp = addressCache[name];
        // false if our cache is invalid or if the resolver doesn't have the required address
        if (resolver.getAddress(name) != cache_tmp || address(0) == cache_tmp) {
            return false;
        }
    }
    return true;
}

But for rebuildCache what are the expected mitigation steps ?