Skip to content
Request an audit

‹ Findings database

OstiumTrading::topUpCollateral() is missing pairsStored.groupMaxCollateral(pairIndex) check

MediumOstium audit·Three Sigma·RWA perpetuals DEX·19th February, 20243S-OS-M03

Description

When opening trades it is checked if the collateral is within limits for each pair in OstiumTradingCallbacks::withinExposureLimits().

The same should be done in OstiumTrading::topUpCollateral().

Recommendation

Add the check to OstiumTrading::topUpCollateral().

Status

Addressed.

Source

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