Beginner FriendlySolidity
100 EXP
View results
Submission Details
Severity: low
Invalid

NFT Burning Instead of Transfer

Summary

The buyOutEstateNFT function burns the NFT instead of transferring it to the buyer.

Vulnerability Details

After payments, nft.burnEstate(_nftID) destroys the NFT, contradicting the intended ownership transfer.

Impact

The buyer gains no NFT ownership, undermining the asset settlement process.

Tools Used

Manual review.

Recommendations

Transfer the NFT to the buyer:

nft.safeTransferFrom(address(this), msg.sender, _nftID);
Updates

Lead Judging Commences

0xtimefliez Lead Judge 6 months ago
Submission Judgement Published
Invalidated
Reason: Design choice

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.