Security context

What an agent needs to avoid regressing past fixes and find the next vuln in this repo.

enzymefinance/protocol

              dev @ da3b870
            

Fixes

CVEs

HIGH

Peak severity

Highlights

Auth Bypass: 34 prior fixes. Scrutinize any change in this area.

contracts/release/core/fund/comptroller/ComptrollerLib.sol: most-fixed (4 issues). Treat as high-risk during review.

56 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: Arbitrary callers could bypass authentication checks and spoof the `msg.sender` as a valid ComptrollerProxy, executing unauthorized operations through the integration manager. Secure integration calls require explicit host allowlisting and adapter registration validations.

Auth Bypass: Unchecked input arguments and raw contract calls allowed users to execute arbitrary contract calls and bypass exchange selector checks when utilizing the deposit wrapper wrapper features. Strictly validating destination targets against an on-chain address registry is critical.

Oracle Manipulation: Failing to validate the success status (e.g., GAV validity flags or stale/corrupted asset prices) returned by the ValueInterpreter allows users to perform atomic actions like purchasing shares using manipulated or stale rates.