Notional Audit
Audit · firm’s term: audit
OpenZeppelin reviewed Notional Finance (Notional Audit) on 22 Dec 2020: 39 findings, 3 critical; 20 marked fixed, 19 open.
Report hosted on blog.openzeppelin.com.
Trustblock does not review code.
Findings39
Statuses are the security firm’s own, from its report.
| Severity | Open | Acknowledged | Fixed |
|---|---|---|---|
| Critical | 0 | 0 | 3 |
| High | 1 | 0 | 4 |
| Medium | 8 | 0 | 6 |
| Low | 10 | 0 | 7 |
High[H01] Uniswap-Chainlink rate difference always reverts for certain currenciesOpen
The report gives no description here.
Medium[M01] contracts addresses in Governed and Directory can get out of syncOpen
The report gives no description here.
Medium[M02] Adding new variables to multi-level inherited upgradeable contracts may break storage layoutOpen
The report gives no description here.
Medium[M02] Casting between types without overflow checksOpen
The report gives no description here.
Medium[M03] Lack of event emission after sensitive actionsOpen
The report gives no description here.
Medium[M04] Contracts storage layout can get corrupted on upgradeable contractsOpen
The report gives no description here.
Medium[M05] Invalid transaction fee encoding specificationsOpen
The report gives no description here.
Medium[M07] liquidate can leave accounts undercollateralized despite assets being availableOpen
The report gives no description here.
Medium[M08] Loops may consume too much gasOpen
The report gives no description here.
Low[L02] Incomplete parameters in emitted eventsOpen
The report gives no description here.
Show 9 more
Low[L03] Inconsistent and incomplete core contracts initializationOpen
The report gives no description here.
Low[L04] Lack of input validationOpen
The report gives no description here.
Low[L05] Missing error messages in require statementsOpen
The report gives no description here.
Low[L06] Multiple conditions in a single require statementOpen
The report gives no description here.
Low[L07] Not using SafeMath functionsOpen
The report gives no description here.
Low[L08] Setting ownerships directly rather than via APIOpen
The report gives no description here.
Low[L09] Unbounded array lengths could lead to out of gas errorsOpen
The report gives no description here.
Low[L10] Casting between types without overflow checksOpen
The report gives no description here.
Low[L11] Unused structOpen
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 (19)
- contracts/utils/Governed.sol
- contracts/ERC1155Token.sol
- contracts/utils/RiskFramework.sol
- contracts/storage/EscrowStorage.sol
- contracts/storage/PortfoliosStorage.sol
- contracts/lib/SafeInt256.sol
- contracts/Escrow.sol
- contracts/utils/Common.sol
- contracts/utils/MockLiquidation.sol
- contracts/interface/IERC1155TokenReceiver.sol
- contracts/interface/IERC165.sol
- contracts/utils/Liquidation.sol
- contracts/CashMarket.sol
- contracts/ERC1155Trade.sol
- contracts/Portfolios.sol
- contracts/utils/ExchangeRate.sol
- contracts/interface/IAggregator.sol
- contracts/interface/IERC777.sol
- contracts/utils/Directory.sol
Part of Notional Finance’s security record
Notional FinanceSecurity firm
OpenZeppelin67 projects reviewed · 184 reports · latest Aug 2026
See OpenZeppelin’s auditsSomething 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.
Tell us