At the request of Dean Eigenmann, I reviewed the following contracts in the upcoming ENS migration:
- ENSRegistry.sol
- ENSRegistryWithFallback.sol
No problems were found in the ENSRegistry.sol
contract, and one mistake was found and fixed in ENSRegistryWithFallback.sol
.
function _setOwner(bytes32 node, address owner) internal {
address addr = owner;
if (addr == address(0x0)) {
addr = address(this);
}
super._setOwner(node, owner);
// ^--- should be addr
}
To the best of my knowledge, the remainder of the contracts work as intended.
0x57f1887a8BF19b14fC0dF6Fd9B2acc9Af147eA85 david2778h.com