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.
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.
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.
manual review
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.
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
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.