maxdao

In the contract `MaxDAO` the role `_owner` has authority over the functions shown in the diagram below.  --- In the contract `Ownable` the role `_owner` has authority over the functions shown in the diagram below.  Any compromise to the privileged accounts may allow the hacker to take advantage of this authority, update the sensitive settings, and execute sensitive functions of the project.

In the contract `MDC` the role `_allowMint` has authority over the functions shown in the diagram below.  --- In the contract `MDC` the role `_owner` has authority over the functions shown in the diagram below.  --- In the contract `Ownable` the role `_owner` has authority over the functions shown in the diagram below.  Any compromise to the privileged accounts may allow the hacker to take advantage of this authority, update the sensitive settings, and execute sensitive functions of the project.

Here is the description text of a finding from that project that might be relevant. It is highly recommended NOT to include OpenZeppelin library code directly in the source code, but import the original library to minimize risk because even slight changes to the library code may lead to critical/major vulnerabilities/bugs. For Solidity version 0.6.x, the latest OpenZeppelin version 3.4.2 should be used.

In the function `depositAlone()`, `currentValue` is calculated with the unit of the token, which is hardcoded as `1e18`. Considering `_token` may have decimal precision other than 18 digits, therefore, the actual unit decimal of the token in `_token` address would be inconsistent with the 1e18.