Skip to content

Commit

Permalink
Improve recommendation on ERC721 transferFrom vs safeTransferFrom (#3647
Browse files Browse the repository at this point in the history
)

Co-authored-by: Francisco Giordano <frangio.1@gmail.com>
  • Loading branch information
goncaloMagalhaes and frangio committed Aug 29, 2022
1 parent 93bc3b6 commit 4b16e88
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion contracts/token/ERC721/IERC721.sol
Original file line number Diff line number Diff line change
Expand Up @@ -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:
*
Expand Down

0 comments on commit 4b16e88

Please sign in to comment.