NFTXFeeDistributor all setter functions (setTreasuryAddress, ...)
NFTXVaultUpgradeable.setManager
Impact
Some wallets still default to zero addresses for a missing input which can lead to breaking critical functionality like setting the manager to the zero address and being locked out.
Handle
@cmichelio
Vulnerability details
Vulnerability Details
Missing parameter validation for functions:
NFTXEligiblityManager.addModule, updateModule
NFTXFeeDistributor
allsetter
functions (setTreasuryAddress
, ...)NFTXVaultUpgradeable.setManager
Impact
Some wallets still default to zero addresses for a missing input which can lead to breaking critical functionality like setting the manager to the zero address and being locked out.
Recommended Mitigation Steps
Validate the parameters.