Skip to content
Request an audit

‹ Findings database

Add a 0 address check on the pool when adding liquidity for greater verbosity

Low/InfoNftperp Matching Engine audit·Three Sigma·NFT perpetuals·29th January, 20243S-NFTPerp-N07

Description

AmmRouter:addLiquidity() does not have an explicit check regarding the existence of the pool with the given id, making it only revert when calling poolsAdded[i].addLiquidity() due to internal EVM checks.

Recomendation

Place an explicit check for this situation.

Status

Addressed in #51fb761

Source

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