Omniscia Trustworks Audit

Manual Review

Manual Review

A thorough line-by-line review was conducted on the codebase to identify potential malfunctions and vulnerabilities in Trustwork's contracts.

As the project at hand implements a token sale and multisignature wallet, 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.

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 certain re-entrancy vulnerabilities within the system which could have had severe ramifications to its overall operation if the owners of the token act maliciously, however, they were conveyed ahead of time to the Trustworks 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 the require checks of the codebase as detailed in the code style chapter.

A total of 12 findings were identified over the course of the manual review of which 4 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
MUL-01MMinorYesConvoluted Logic
MUL-02MMinorYesPotentially Breaking Functionality
TAP-01MMinorYesInapplicacy of Checks-Effects-Interactions Pattern
TAP-02MMinorYesPotentially Breaking Functionality