Omniscia Pirex Audit

Manual Review

Manual Review

A thorough line-by-line review was conducted on the codebase to identify potential malfunctions and vulnerabilities in the Pirex multi-token Convex integrating codebase.

As the project at hand implements a complex Convex-integrating wrapper protocol, intricate care was put into ensuring that the flow of funds within the system conforms to the specifications and restrictions laid forth within the protocol's specification as well as Convex's documentation and deployed code.

We validated that all state transitions of the system occur within sane criteria and that all rudimentary formulas within the system execute as expected. We pinpointed multiple important flaws within the system which could have had severe ramifications to its overall operation, however, they were conveyed ahead of time to the Pirex team to be promptly remediated.

Additionally, the system was investigated for any other commonly present attack vectors such as re-entrancy attacks, mathematical truncations, logical flaws and ERC / EIP standard inconsistencies. The documentation of the project was satisfactory to a certain extent, however, we strongly recommend the documentation of the project to be expanded at certain complex points such as the in-line description of each of the project's token types as well as standardized documentation of external-facing functions.

A total of 24 findings were identified over the course of the manual review of which 17 findings concerned the behaviour and security of the system. The non-security related findings, such as optimizations, are included in the separate Code Style chapter.

The finding table below enumerates all these security / behavioural findings:

IDSeverityAddressedTitle
ERP-01MMisleading Contract Documentation
ERS-01MInsecure Elliptic Curve Recovery Mechanism
PCV-01MIncorrect Proportionate Claim Mechanism
PCV-02MImproper Prevention of Emergency Migration
PCV-03MMismatch of Maximum Redemption Time
PCV-04MDisassociation of Votium Time Rewards
PCV-05MImproper Sanitization of New Fee
PCV-06MInexistent Migration Delay
PCV-07MInexistent Prevention of Duplicate Entries
PCV-08MPotentially Harmful Epoch Rounding
PCV-09MUnaudited External Contract Interaction
PCC-01MImproper Adjustment of Delegation Space & Registry
PCC-02MIncorrect Usage of Funds at Rest
PCC-03MUnlimited CVX Allowance Centralization
PCC-04MUnsafe & Inefficient Type
PFS-01MImproper & Inefficient Calculation of Fees
PCX-01MSnapshot Initialization Race Condition