Lucene search

K
code423n4Code4renaCODE423N4:2022-09-ARTGOBBLERS-FINDINGS-ISSUES-352
HistorySep 27, 2022 - 12:00 a.m.

Use safeTransferFrom instead of transferFrom for ERC721 transfers

2022-09-2700:00:00
Code4rena
github.com
4
vulnerability
impact
nfts
erc721
transfer
onerc721received
safetransferfrom
solidity
vscode
mitigation

Lines of code

Vulnerability details

Impact

Any NFTs can be transferred here, there are a few NFTs (here’s an example) that have logic in the onERC721Received() function, which is only triggered in the safeTransferFrom() function and not in transferFrom().

Tools Used

Solidity Visual Developer of VSCode

Recommended Mitigation Steps

Recommend using safeTransferFrom().


The text was updated successfully, but these errors were encountered:

👎 1 Shungy reacted with thumbs down emoji

All reactions

  • 👎 1 reaction