Skip to content
Request an audit

‹ All findings

StakingOperator::setCollateralToken() will cause issues if it changes tokens relations that have already been set

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

Description

Tokens are locked and unlocked, minting and burning zkTokens in the process, respectively. Thus, if zkTokens have been minted, but then the mappings between original and collateral tokens are changed, it will burn other zkTokens, leading to issues.

Recommendation

Either place an explicit check in the function to ensure the mappings have not been already created or keep this in mind.

Status

Addressed in #6e589cb, #dc4f4a4.