<!-- canonical: https://0xsimao.com/findings/singularity-variables-initialized-default -->

# Variables are initialized to 0 by default

Low/Info · Three Sigma · Private DeFi access · 26th February, 2024

Finding 3S-SG-N05 of the Singularity security review.

- Protocol: https://www.thesingularity.network/
- Report: /reports/singularity
- Source: https://cdn.sanity.io/files/qoqld077/production/45b8aac56a2b8e3b557df4329ac9bf8220b64012.pdf

---

### Description

Some variables such as [these](https://github.com/portalgateme/darkpool-v1-zk-contracts-fork/blob/889667f80217ebe2788cba40bebda18a1ef13ecb/contracts/defi/curve/CurveMPAddLiquidityAssetManager.sol#L197-L198) are initialized to 0, which is not necessary.

### Recommendation

Consider not initializing the variables as it is not required.

### Status

Addressed in [#8b300f0](https://github.com/portalgateme/darkpool-v1-zk-contracts-fork/commit/8b300f07a28f2c073154c8de3e9e90ba64ef1934).
