Raisebox Faucet

First Flight #50
Beginner FriendlySolidity
100 EXP
View results
Submission Details
Impact: low
Likelihood: low
Invalid

[L-8] State Change Without Event

[L-8] State Change Without Event

Description

  • Normal behavior: Emit events for significant state changes to aid off-chain indexing.

  • Specific issue: adjustDailyClaimLimit updates dailyClaimLimit without emitting an event.

function adjustDailyClaimLimit(uint256 by, bool increaseClaimLimit) public onlyOwner { ... } // @> no event

Risk

Likelihood:Low

  • Owner will change limits during maintenance; off-chain systems will miss changes without events.

Impact:Low

  • Minor: reduced transparency and harder monitoring.

Proof of Concept

// Call adjustDailyClaimLimit and no event is emitted.

Recommended Mitigation

+ event DailyClaimLimitUpdated(uint256 oldLimit, uint256 newLimit);
function adjustDailyClaimLimit(uint256 by, bool increaseClaimLimit) public onlyOwner {
uint256 old = dailyClaimLimit;
// ... update logic ...
emit DailyClaimLimitUpdated(old, dailyClaimLimit);
}
Updates

Lead Judging Commences

inallhonesty Lead Judge 5 days ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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