From 4b16e887474eebb64e3f8266c68ef151a3f1b769 Mon Sep 17 00:00:00 2001 From: "gmhacker.eth" <95878230+goncaloMagalhaes@users.noreply.github.com> Date: Mon, 29 Aug 2022 21:40:21 +0100 Subject: [PATCH] Improve recommendation on ERC721 transferFrom vs safeTransferFrom (#3647) Co-authored-by: Francisco Giordano --- contracts/token/ERC721/IERC721.sol | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/contracts/token/ERC721/IERC721.sol b/contracts/token/ERC721/IERC721.sol index 4bb973b3e..46f73b4ef 100644 --- a/contracts/token/ERC721/IERC721.sol +++ b/contracts/token/ERC721/IERC721.sol @@ -81,7 +81,9 @@ interface IERC721 is IERC165 { /** * @dev Transfers `tokenId` token from `from` to `to`. * - * WARNING: Usage of this method is discouraged, use {safeTransferFrom} whenever possible. + * WARNING: Note that the caller is responsible to confirm that the recipient is capable of receiving ERC721 + * or else they may be permanently lost. Usage of {safeTransferFrom} prevents loss, though the caller must + * understand this adds an external call which potentially creates a reentrancy vulnerability. * * Requirements: *