Highlights
Auth Bypass: 6 prior fixes. Scrutinize any change in this area.
contracts/src/DataHavenServiceManager.sol: most-fixed (3 issues). Treat as high-risk during review.
7 high-severity fixes in this history; regressions here are high-impact.
Recurring patterns
The bug types that recur here, drawn from past fixes, not open vulnerabilities.
Auth Bypass: Unauthorized actors could previously trigger operator de-registrations because vital administrative functions lacked proper caller verification modifiers. Future modifications to validator and operator states must enforce strict authorization checks.
Auth Bypass: Inbound bridge queues previously updated the active validator set without verifying that the messages originated exclusively from the trusted DataHavenServiceManager Ethereum address. This allowed arbitrary bridge messages to execute high-privilege configuration operations.
Access Control: Replay attacks and out-of-order execution were possible on validator updates and rewards submission functions. Implementing sequential era validation and replay guard mappings are required to prevent malicious double-submission of state updates.