Skip to content
Request an audit

‹ All findings

Duplicated slippage check in SyrupUserActions

Low/InfoMaple Syrup·Three Sigma · Permissionless lending · 23rd August, 20243S-SY-N02

Description

SyrupUserActions::_swap() checks the slippage at the end require(amountOut_ >= minAmountOut_, "SUA:S:INSUFFICIENT_AMOUNT_OUT");. When the asset out is USDC, it double checks the slippage in SyrupUserActions::_swapDaiForUsdc() as require(usdcOut_ >= minUsdcOut_, "SUA:SDU:INSUFFICIENT_AMOUNT_OUT");.

Recommendation

There is no need to check the slippage in SyrupUserActions::_swapDaiForUsdc().