NoBey/openzeppelin-contracts
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Update ERC721.sol

Browse Source
This commit is contained in:
Hadrien Croubois
2023-07-14 15:31:29 +02:00
committed by GitHub
parent e26d5c0951
commit 2897abccc9
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
contracts/token/ERC721/ERC721.sol
View File

@ -192,10 +192,7 @@ abstract contract ERC721 is Context, ERC165, IERC721, IERC721Metadata, IERC721Er
* @dev Returns whether `spender` is allowed to manage `owner`'s tokens, or `tokenId` in
* particular (ignoring whether it is owned by `owner`).
*
* WARNING: This function doesn't check that `owner` is the actual owner of the specified
* `tokenId`. Moreover, it returns true if `owner == spender` in order to skip the check where
* needed. Consider checking for cases where `spender == address(0)` since they could lead to
* unexpected behavior.
* WARNING: This function doesn't check that `owner` is the actual owner of `tokenId`.
*/
function _isAuthorized(address owner, address spender, uint256 tokenId) internal view virtual returns (bool) {
return
@ -208,7 +205,7 @@ abstract contract ERC721 is Context, ERC165, IERC721, IERC721Metadata, IERC721Er
* Reverts if `spender` has not approval for all assets of the provided `owner` nor the actual owner approved the `spender` for the specific `tokenId`.
*
* WARNING: This function relies on {_isAuthorized}, so it doesn't check whether `owner` is the
* actual owner of `tokenId` and will skip the check if `spender == owner`
* actual owner of `tokenId`.
*/
function _checkAuthorized(address owner, address spender, uint256 tokenId) internal view virtual {
if (!_isAuthorized(owner, spender, tokenId)) {