Lack of address validation in MasterDeployer.addToWhitelist
#138
Labels
0 (Non-critical)
Code style, clarity, syntax, versioning, off-chain monitoring (events etc), exclude gas optimisation
bug
Warden finding
sponsor acknowledged
Technically the issue is correct, but we're not going to resolve it for XYZ reasons
Handle
GalloDaSballo
Vulnerability details
Impact
addToWhitelist
https://github.com/sushiswap/trident/blob/9130b10efaf9c653d74dc7a65bde788ec4b354b5/contracts/deployer/MasterDeployer.sol#L50
Is not checking for
_factory
being different from address(0)Recommended Mitigation Steps
Add
require(_factory != address(0))
The text was updated successfully, but these errors were encountered: