Submitted by 0x29A, also found by Amithuddar, benbaessler, berndartmueller, cccz, CRYP70, rbserver, RedOneN, and Sm4rty
NameWrapper.sol#L327-L346
The unwrapETH2LD use transferFrom to transfer ERC721 token, the newRegistrant could be an unprepared contract.
Should a ERC-721 compatible token be transferred to an unprepared contract, it would end up being locked up there. Moreover, if a contract explicitly wanted to reject ERC-721 safeTransfers.
Plus take a look to the OZ safeTransfer comments:
Usage of this method is discouraged, use safeTransferFrom whenever possible.
jefflau (ENS) disputed and commented:
LSDan (judge) commented:
jefflau (ENS) commented:
