Pieces Protocol

First Flight #32
Beginner FriendlyFoundrySolidityNFT
100 EXP
View results
Submission Details
Severity: low
Invalid

Lack of Events for Important State Changes

Summary:Some critical state changes do not emit events.

Vulnerability Details:

function transferErcTokens(address nftAddress, address to, uint256 amount) external {
// ...
balances[msg.sender][tokenInfo.erc20Address] -= amount;
balances[to][tokenInfo.erc20Address] += amount;
}

Impact:Difficulty in off-chain tracking and verification.

Tools Used:Remix IDE

Recommendations:Emit events for all significant state changes to improve transparency and off-chain tracking.

event TokensTransferred(address indexed from, address indexed to, address indexed erc20Address, uint256 amount);
Updates

Lead Judging Commences

fishy Lead Judge 8 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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