Vanguard

First Flight #56
Beginner FriendlyDeFiFoundry
0 EXP
Submission Details
Impact: high
Likelihood: high

Penalty Fees Are Not Accounted or Redirected

Author Revealed upon completion

Fee Effects are not transparent as the penalty fees are not accounted or redirected

Description

  • When penalties are applied, higher swap fees should either discourage behavior or transparently redistribute value (e.g., to LPs or the protocol).

  • Although the contract tracks totalPenaltyFeesCollected, this variable is never updated. Penalty fees are implicitly absorbed by Uniswap’s fee mechanism without explicit accounting or attribution, resulting in misleading state and unclear economic outcomes.

uint256 public totalPenaltyFeesCollected;
// @> Variable is never incremented or used

Risk

Likelihood:

  • Occurs whenever penalty fees are applied

  • Affects all penalized swaps

Impact:

  • Misleading accounting state

  • Auditors and integrators may assume penalties are tracked when they are not

  • Economic behavior becomes opaque

Proof of Concept

  1. Swap violates phase conditions

  2. Penalty fee override is applied

  3. totalPenaltyFeesCollected remains unchanged

  4. No mechanism exists to reconcile penalty flow

Support

FAQs

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

Give us feedback!