Fei Protocol v2 Phase 1

Consensys1 Sep 2021Files named in the report

Consensys reviewed Fei Protocol (Fei Protocol v2 Phase 1) on 1 Sep 2021: 21 findings, 0 critical; none marked fixed.

No open critical findings

Open the report Open Fei Protocol’s security record

Report hosted on diligence.security.

Trustblock does not review code.

Findings21

Statuses are the security firm’s own, from its report.

Severity	Open	Acknowledged	Fixed
Critical	0	0	0
High	2	0	0
Medium	6	0	0
Low	13	0	0

HighCollateralizationOracle - Fei in excluded deposits contributes to userCirculatingFeiOpen

The report gives no description here.

HighStableSwapOperatorV1 - resistantFei value is not correct in the resistantBalanceAndFei functionOpen

The report gives no description here.

MediumBalancerLBPSwapper - init() can be front-run to potentially steal tokensOpen

The report gives no description here.

MediumChainlinkOracleWrapper - latestRoundData might return stale resultsOpen

The report gives no description here.

MediumCollateralizationOracleWrapper - the deviation threshold check in update() always returns falseOpen

The report gives no description here.

MediumPCVEquityMinter and BalancerLBPSwapper - desynchronisation raceOpen

The report gives no description here.

MediumStableSwapOperatorV1 - FEI tokens in the contract are not considerred as protocol-ownedOpen

The report gives no description here.

MediumStableSwapOperatorV1 - the _minLpOut value is not accurateOpen

The report gives no description here.

LowBalancerLBPSwapper - tokenSpent and tokenReceived should be immutableOpen

The report gives no description here.

LowCollateralizationOracle - misleading commentsOpen

The report gives no description here.

Show 10 more

LowCollateralizationOracle - missing events and incomplete event informationOpen

The report gives no description here.

LowCollateralizationOracle - potentially unsafe castsOpen

The report gives no description here.

LowCollateralizationOracle - swapDeposit should call internal functions to remove/add depositsOpen

The report gives no description here.

LowFeiTimedMinter - constructor does not enforce the same boundaries as setter for frequencyOpen

The report gives no description here.

LowNo emergency exit strategy for BalancerLBPSwapperOpen

The report gives no description here.

LowRateLimited - Contract starts with a full buffer at deploymentOpen

The report gives no description here.

LowRe-initialization of the Balancer pool is potentially possibleOpen

The report gives no description here.

LowStableSwapOperatorV1 - the contract relies on the 1$ price of every token in 3poolOpen

The report gives no description here.

LowThe BalancerLBPSwapper may not have enough Tribe tokensOpen

The report gives no description here.

LowTribeReserveStabilizer - different minting proceduresOpen

The report gives no description here.

Show 1 more

LowUpdate NatspecOpen

The report gives no description here.

Contracts in scope

No deployed address is recorded for this audit.

The report names a repository or files. Open the report to see its scope.

Repository and files the report names (27)

contracts/oracle/CollateralizationOracleWrapper.sol
contracts/pcv/balancer/manager/WeightedBalancerPoolManager.sol
contracts/oracle/ICollateralizationOracleWrapper.sol
contracts/token/IFeiTimedMinter.sol
contracts/utils/RateLimitedMinter.sol
contracts/pcv/utils/ERC20Splitter.sol
contracts/Constants.sol
contracts/oracle/ICollateralizationOracle.sol
contracts/pcv/utils/PCVDepositWrapper.sol
contracts/oracle/CollateralizationOracle.sol
contracts/pcv/balancer/IAssetManager.sol
contracts/token/FeiTimedMinter.sol
contracts/stabilizer/TribeReserveStabilizer.sol
contracts/utils/RateLimited.sol
contracts/token/IPCVEquityMinter.sol
contracts/pcv/balancer/IBasePool.sol
contracts/token/PCVEquityMinter.sol
contracts/pcv/balancer/IWeightedPool.sol
contracts/oracle/ConstantOracle.sol
contracts/pcv/balancer/manager/IBaseBalancerPoolManager.sol
contracts/pcv/balancer/manager/IWeightedBalancerPoolManager.sol
contracts/keeper/CollateralizationOracleKeeper.sol
contracts/pcv/balancer/BalancerLBPSwapper.sol
contracts/pcv/balancer/manager/BaseBalancerPoolManager.sol
contracts/pcv/balancer/IVault.sol
contracts/pcv/utils/StaticPCVDepositWrapper.sol
contracts/pcv/curve/StableSwapOperatorV1.sol

Part of Fei Protocol’s security record

Fei Protocol

Audits5 audits by 2 firmsLast audit Sep 2021, by Consensys.

Critical findings1 open critical findingReported in “Fei Tribechief” by Consensys, Jul 2021. Not marked fixed in that report.

Open Fei Protocol’s security record

Security firm

Consensys

41 projects reviewed · 59 reports · latest Oct 2024

See Consensys’s audits

Something wrong on this page? A missing report, a wrong date, a finding that was fixed since. Tell us and we check it against its source.