Read the findings of the 2022 Web3 Security Report

In the contract `Generation` the role `_owner` has authority over the functions shown in the diagram below.
Any compromise to the `_owner` account may allow a hacker to take advantage of this authority. For example, the owner would be able to lock the tokens of any user for an unlimited amount of time, stop users from transferring tokens, and mint any number of tokens. 

![](https://accelerator-tasks-prod.s3.amazonaws.com/040b983143ec46f78e7aa530e31abf98/diagrams/centralization_Generation__owner.svg)


Centralization Risks in Generation.sol

The logic of the functions `lockAfter()` and `transferWithLockAfter()`'s implementation is similar to the logic in functions `lock()` and `transferWithLockAfter()` respectively. Therefore, there's chance to improve the reusability of the project by invoking functions `lock()` and `transferWithLockAfter()` directly.

Possible Reusability Improvement

The solidity versions (released before 0.8.0) do not have integrated SafeMath. Therefore there is no prevention for overflow and underflow errors. It is advisable replace the selected arithmetic operations for functions in SafeMath library. 

Usage of SafeMath library

One or more 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


Declaration Naming Convention

Solidity frequently releases new compiler versions. Using an old version prevents access to new Solidity security features. Also, recent versions may be too early to be trusted. 


Solidity Version Not Recommended

Literals with many digits are difficult to read and review.

File: Generation.sol (Line 284, Function `Generation.slitherConstructorConstantVariables`)
```solidity
    uint256 public constant initialSupply = 10000000000 * (10 ** uint256(decimals));
```



Too Many Digits

Using `now` can be risky since timestamps can be influenced by miners. That means the miner creating the block can manipulate to change the outcome of the transaction when they are used for generating randomness.

File: Generation.sol (Line 471, Function `Generation.releaseLock`)
```solidity
            if (lockInfo[_holder][i].releaseTime <= now) {
```



Usage of `block.timestamp`

`public` functions that are never called by the contract could be declared as `external`. `external` functions are more efficient than `public` functions.

Improper Usage of `public` and `external` Type

The **require** can be used to check for conditions and throw an exception if the condition is not met. It is better to provide a string message containing details about the error that will be passed back to the caller.

Missing Error Messages

The linked statements do not affect the functionality of the codebase and appear to be either leftover from test code or older functionality.
`_burnFrom' burns the sender's allowance tokens of the said account. If the sender's amount is not within the allowance of the token holder, leading to overflow attacks.


Redundant Code Components

It is advisable to check the standard libraries for changes/updates in the content. The `mod()` function has been removed from the SafeMath library. You can find `mod()` in the Math library.

Openzeppelin Library Usage

Generation

Generation.sol

 Generation 

Phishing scammer Monkey Drainer has pilfered as much as $1M in Ethereum

Did Mark Zuckerberg’s Little Dance Actually Show Us Real Metaverse Legs? | Transparent NEWS

Ethereum (ETH) Sees High Volatility as U.S. Records Higher-Than-Expected GDP Growth

Apple Wants to Play in the Metaverse. Don't Expect It to Be Easy.

BTC price struggles at $21K as trader says ‘top is in’ for Bitcoin, Ethereum

An auditor in the metaverse. No proper tracking of cash or employees. New FTX CEO describes Sam Bankman-Fried’s haphazard management.

Cardano Founder Makes Staggering Prediction about BTC, XRP May Make Everyone Cry, SHIB Metaverse Team Reveals Crucial Update: Crypto News Digest by U.Today

Generation

Highlights & Alerts

This is just a placeholder

highlight placeholder lorem Ipsum...

highlight lorem

This is a alert demo

Alert demo

Code Audit History

11

0

11

0

0