Skip to content
Request an audit

‹ Findings database

Uniswap collect fees should skip collecting fees of one of the tokens if the amount is 0

Low/InfoSingularity audit·Three Sigma·Private DeFi access·26th February, 20243S-SG-L08

Description

Uniswap positions may collect fees in only of the tokens, leading to 0 fees in the other. This will cause the fee manager to revert in calculateFee().

Recomendation

Skip transferring the funds if the amount is 0.

Status

Addressed in #8b300f0.

Source

Disclosed by 0xSimao · Sherlock's 2025 Watson of the Year · 21 first places in 57 audit contests