Eggstravaganza

First Flight #37
Beginner FriendlySolidity
100 EXP
View results
Submission Details
Severity: low
Invalid

Missing event for function that change critical state

Summary

Function setEggFindThreshold() change eggFindThreshold value, but does not emit event

Vulnerability Details

Events allow capturing the changed parameters so that off-chain tools/interfaces can register such changes that allow users to evaluate them. Missing events do not promote transparency and if such changes immediately affect users’ perception of fairness or trustworthiness, they could exit the protocol causing a reduction in protocol users.

Owner can change eggFindThreshold value, using function setEggFindThreshold() in EggHuntGame contract. Function does not emit event after changing value.

Impact

User could not detect changing important value for them by filtering events, because function does not emit events.

Tools Used

Manual reveiw

Recommendations

Add event for function, which change important storage value

Updates

Lead Judging Commences

m3dython Lead Judge about 2 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

Event Emission

Standard practice for clarifying important contract behaviors

Support

FAQs

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