Liquidity Management
Gamma
DeFiFoundry
50,000 USDC
View results
Previous Next
Submission Details
Severity: low
Invalid

Unused COMPOUND_ACTION NextActionSelector enum variant.

tadev

Summary

NextActionSelector enum defines a COMPOUND_ACTION enum but it is never used. This variant can safely be removed.

enum NextActionSelector {
NO_ACTION, // state where no action is required
INCREASE_ACTION, // represents increasing actions such as increasing a GMX perps position or increasing a spot position
SWAP_ACTION, // represents swapping a token, either collateral to index or vice versa
WITHDRAW_ACTION, // represents withdrawing funds
//@audit: COMPOUND_ACTION variant never used
COMPOUND_ACTION, // whenver claiming positive funding fee, we could have idle funds in the vault.
// If it's enough to deposit more into GMX, triggers `COMPOUND_ACTION`.
SETTLE_ACTION, // settle fees and ensure state is up-to-date prior to withdrawals
FINALIZE
}

Impact

The impact of this issue is low/informationnal.

Tools Used

Manual review.

Recommendations

Remove the unused enum variant.

Updates

Lead Judging Commences

n0kto Lead Judge 5 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

Informational or Gas

Please read the CodeHawks documentation to know which submissions are valid. If you disagree, provide a coded PoC and explain the real likelihood and the detailed impact on the mainnet without any supposition (if, it could, etc) to prove your point.

n0kto Lead Judge 5 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

Informational or Gas

Please read the CodeHawks documentation to know which submissions are valid. If you disagree, provide a coded PoC and explain the real likelihood and the detailed impact on the mainnet without any supposition (if, it could, etc) to prove your point.

Support

FAQs

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