DeFiLayer 1Layer 2
14,723 OP
View results
Submission Details
Severity: low
Valid

Missing Event Emission

Summary

The contract lacks event emissions for successful updates, such as changes to the profit_max_unlock_time parameter. While this omission does not directly compromise security, it reduces transparency by making it harder to monitor and audit critical state changes in the contract.

Vulnerability Details

The contract does not emit any events when updates to the profit_max_unlock_time parameter are made. This means that whenever this parameter is modified, there is no on-chain log that records the change.

Impact

Transparency Loss: Without event logs, external monitoring systems, auditors, and users cannot easily track changes to this critical parameter.

  • Auditing Difficulties: The absence of events makes forensic analysis and real-time auditing of parameter changes more challenging.

  • Security Implications: Although this vulnerability does not directly compromise the security or functionality of the contract, it can obscure important state changes that might be relevant during incident response or investigation.

Tools Used

manual review

Recommendations

  • To enhance transparency and ease monitoring, it is recommended to emit an event (e.g., ProfitMaxUnlockTimeUpdated) every time the profit_max_unlock_time parameter is updated. This would help in maintaining a reliable audit trail and improve the contract's overall observability.

Updates

Lead Judging Commences

0xnevi Lead Judge 5 months ago
Submission Judgement Published
Validated
Assigned finding tags:

finding-missing-event-emission-profit_max_unlock_time

I believe low severity to be appropriate here, events could be used to be more explicit for the upcoming effects on price for changes in max unlock time, similar to as included in `update_price` with the `PriceUpdate` event

Support

FAQs

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