Skip to content
Request an audit

‹ All findings

Fees or vested position withdrawal in ShadowStrategyGauge::addVestingPosition() when collecting fees will be revested

MediumYieldoor Gauges·0xSimao · Leveraged yield farming · 12th May 2025M-1

ReportSource

Description:

ShadowStrategyGauge::_addVestingPosition() collects fees first, coming either from LP fees or withdrawing from the vested position. Any leftover balance exceeding the initial balance after this call in ShadowStrategyGauge::collectGaugeRewards() is considered vested position, but this may result from already unvested amounts or claimed fees, which will be vested again.

Recommended Mitigation:

Collect fees at the beginning of the ShadowStrategyGauge::collectGaugeRewards() call, before the balance snapshot is taken.