Submission Details
Impact:
Likelihood:
Missing Events for Critical State Changes
Description:
Several important functions do not emit events:
cancelParticipation()- no event emitted_getWinnerShares()- state change not logged_setFinallizedVaultBalance()- state change not logged
Events are important for off-chain tracking and transparency.
Impact:
Difficult to track state changes off-chain
Poor transparency
Harder to build front-end applications
Mitigation:
+ event ParticipationCancelled(address indexed user, uint256 refundAmount);
+ event VaultFinalized(uint256 finalBalance);
+ event WinnerSharesCalculated(uint256 totalWinnerShares);
function cancelParticipation() public {
// ... existing logic ...
+ emit ParticipationCancelled(msg.sender, refundAmount);
}
function _setFinallizedVaultBalance () internal returns (uint256) {
// ... existing logic ...
+ emit VaultFinalized(finalizedVaultAsset);
}
function _getWinnerShares () internal returns (uint256) {
// ... existing logic ...
+ emit WinnerSharesCalculated(totalWinnerShares);
}
Rewards breakdown
nSLOC
193This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.