Token Importer
ZoraVerifier.sol
Verifies tokens being imported via the TokenImporter from the Zora platform.
[INFO] @dev Comment is Incorrect
On lines #43 & #44 the @dev comment mentions that legacy factories cannot be supported, however the setZoraCoinImplementation function would support this. This can just be deleted.
[LOW] Missing Zero Address Check
The setZoraCoinImplementation function allows the owner to add or remove the zero address as a valid implementation. If the memecoin passed doesn't use a proxy implementation, then a zero address will be returned and would incorrectly be considered valid.
We recomment adding a check to prevent the zero address from being added as a valid implementation.
/**
* Sets or removes a Zora coin implementation address.
*
* @param _zoraCoinImplementation The address of the Zora coin implementation
* @param _valid Whether the implementation is valid
*/
function setZoraCoinImplementation(address _zoraCoinImplementation, bool _valid) external onlyOwner {
if (_zoraCoinImplementation == address(0)) {
revert ZeroAddress();
}
// Existing logic
}