Lucyan99 - NC-1: Missing checks for `address(0)` when assigning values to address state variables

[sherlock-admin2]

Lucyan99

medium

NC-1: Missing checks for address(0) when assigning values to address state variables

Summary

Assigning values to address state variables without checking for address(0).

Code Snippet

• Found in contracts/JalaFactory.sol Line: 24

          feeToSetter = _feeToSetter;

• Found in contracts/JalaFactory.sol Line: 61

          feeTo = _feeTo;

• Found in contracts/JalaFactory.sol Line: 67

          feeToSetter = _feeToSetter;

• Found in contracts/JalaPair.sol Line: 89

          token0 = _token0;

• Found in contracts/JalaPair.sol Line: 90

          token1 = _token1;

• Found in contracts/utils/ChilizWrappedERC20.sol Line: 23

          underlyingToken = _underlyingToken;

Recommendation

Revert if the address is address(0).

[nevillehuang]

Invalid based on sherlock rules

3. Zero address checks: Check to make sure input values are not zero addresses.