Browse Source

Improve recommendation on ERC721 transferFrom vs safeTransferFrom (#3647)

Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
(cherry picked from commit 4b16e887474eebb64e3f8266c68ef151a3f1b769)
gmhacker.eth 3 years ago
parent
commit
3a64f471bf
1 changed files with 3 additions and 1 deletions
  1. 3 1
      contracts/token/ERC721/IERC721.sol

+ 3 - 1
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:
      *