Skip to content

Request an audit

‹ All findings

Unnecessary abi.decode in PHyperLPoolSwapInside::_swapWithCustomData()

Low/InfoClip Finance Strategy Upgrade·by the Three Sigma team·DeFi infrastructure·15th March 20243S-Clip-N01

Report Page Source

Description

PHyperLPoolSwapInside::_swapWithCustomData() does abi.decode of the returnData, which is already a bytes variable.

Recommendation

revert(string(returnData)); should be enough

Status

Addressed in #0c8de23.

Found during Three Sigma’s team review · 0xSimao on the review team