Beatland Festival
First Flight #44
Beginner FriendlyFoundrySolidityNFT
100 EXP
View results
Previous Next
Submission Details
Impact: low
Likelihood: high
Invalid

Missing Event Emission after changing state variable `oraganizer` in `FestivalPass`.

0xnigthswatch

Missing Event Emission after changing state variable oraganizer in FestivalPass.

Description

  • In FestivalPass, the function setOrganizer is called by the owner to set an organizer who is reposible on mutiple fucntionalities of this contract. But, this function does not emit an event when the owner set a new organizer.

  • Without events, off-chain systems cannot track these updates efficiently.

// organizer is set without event emission
function setOrganizer(address _organizer) public onlyOwner {
@> organizer = _organizer;
}

Risk

Likelihood:

  • High, setOrganizeris called in the constructor, so it will occur at deployment and every time the owner change to a new organizer.

Impact:

  • Off-chain systems cannot track state changes.

  • Degrades transparency and traceability for users and developers.

Proof of Concept

N/A

Recommended Mitigation

  • Add event to be emitted when a new organzier is created.

+ event OrganizerSetted(address indexed organizer)
​
function setOrganizer(address _organizer) public onlyOwner {
organizer = _organizer;
+ emit OrganizerSetted(organizer);
}
Updates

Lead Judging Commences

inallhonesty Lead Judge 4 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

Missing events / Events not properly configured

Informational. This protocol doesn't rely on events to function, they are just nice to have, but not mandatory.

Support

FAQs

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