Skip to content
Request an audit

‹ Findings database

UniswapLiquidityAssetManager registers the note footer twice

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

Description

The note footers are registered twice. https://github.com/portalgateme/darkpool-v1-zk-contracts-fork/blob/master/contracts/defi/uniswap/UniswapLiquidityAssetManager.sol#L574-L575 https://github.com/portalgateme/darkpool-v1-zk-contracts-fork/blob/master/contracts/defi/uniswap/UniswapLiquidityAssetManager.sol#L532-L533

Recommendation

Only register once.

Status

Addressed in #8b300f0.

Source

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