BriVault

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

The event is not correctly placed by not logging the address correctly.

Root + Impact

The event is not logged properly

Description

  • The event are mainly used to logg data to the front end in an approparaite way by providing the information onchain to the front end

  • The wrong calling of the event may cause the data to not be logged properly to the front end as it doesn't add the keywords necessary to log the data

// Root cause in the codebase with @> marks to highlight the relevant
event joinedEvent (address user, uint256 _countryId);
event Withdraw (address user, uint256 _amount);

Risk

Likelihood:

The likelihood, is low because it the information can be lost


Proof of Concept

Recommended Mitigation

- remove this code
- event joinedEvent (address user, uint256 _countryId);
- event Withdraw (address user, uint256 _amount);
+ add this code
+ event joinedEvent (address indexed user, uint256 _countryId);
+ event Withdraw (address indexed user, uint256 _amount);
Updates

Appeal created

bube Lead Judge 20 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.

Give us feedback!