Merge pull request #168 from lidofinance/fix/indexed-events

Audit fix: Indexed Event Parameters for State Changes
lidofinance · Nov 11, 2024 · 224c082 · 224c082
2 parents 2ece799 + 7104c0d
commit 224c082
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/contracts/libraries/DualGovernanceStateMachine.sol b/contracts/libraries/DualGovernanceStateMachine.sol
@@ -91,7 +91,7 @@ library DualGovernanceStateMachine {
     // ---
 
     event NewSignallingEscrowDeployed(IEscrow indexed escrow);
-    event DualGovernanceStateChanged(State from, State to, Context state);
+    event DualGovernanceStateChanged(State indexed from, State indexed to, Context state);
     event ConfigProviderSet(IDualGovernanceConfigProvider newConfigProvider);
 
     // ---

diff --git a/contracts/libraries/EscrowState.sol b/contracts/libraries/EscrowState.sol
@@ -35,7 +35,7 @@ library EscrowState {
     // Events
     // ---
 
-    event EscrowStateChanged(State from, State to);
+    event EscrowStateChanged(State indexed from, State indexed to);
     event RageQuitExtensionPeriodStarted(Timestamp startedAt);
     event MinAssetsLockDurationSet(Duration newAssetsLockDuration);
     event RageQuitStarted(Duration rageQuitExtensionDuration, Duration rageQuitEthWithdrawalsDelay);