Leaven Network - Audit

The linked declarations do not conform to the [Solidity style guide](https://solidity.readthedocs.io/en/v0.7.3/style-guide.html#naming-conventions) with regards to its naming convention. Particularly: - `camelCase`: Should be applied to function names, argument names, local and state variable names, modifiers - `UPPER_CASE`: Should be applied to `constant` variables - `CapWords`: Should be applied to contract names, struct names, event names and enums

The functions which are never called internally within the contract should have external visibility for gas optimization.

No unit tests were provided for the contracts in scope. Testing programs are a very important aspect of proving program correctness, preventing regressions, and release engineering. Without tests, there is no way to know if the program works as expected. The fact that code was shipped for review with such major flaw, is considered a signal of some problems with release engineering procedures, which may lead to shipping not intended changes to production.

There should always be events emitted in the sensitive functions that are controlled by centralization roles.

State variables that never change should be declared as `constant` to save gas. - `gasprice` should be declared `constant`: ```solidity=1259 address public gasprice = address(0xA43509661141F254F54D9A326E8Ec851A0b95307); ```

There are a few typos in the contract.

- `Deposits` is declared in `LeavenVaultV6` but never emitted: ```solidity=1116 event Deposits(address sender, uint256 amount); ```