Findings

Issue 01

Type

Severity

Location

Status

Owner Capabilities

High

setTaxFee

✔️ Resolved

Description

The owner of the contract can set the tax fee to any value he desires.

Recommendation:

Consider adding an upper limit to the set functions.

Issue 02

Type

Severity

Location

Status

Logical Issue

Low

_transfer

Acknowledged

Description

There is no way to distinguish between buy/sell and add/remove liquidity transactions since both buy and remove liquidity operations receive tokens from the pair address (the pair is sending tokens to the caller) and sell and add liquidity follow the same logic. Therefore, fees will be taken on add liquidity transactions and remove liquidity transactions as well.

Recommendation:

Consider supporting excluding specific addresses from fees (For example, the address which adds the liquidity).

Issue 03

Type

Severity

Location

Status

Volatile Code

High

burn

✔️ Resolved

Description

The Admin role can burn tokens (decrease the balance of any holder) on behalf of any address.

Recommendation:

Consider implementing burnFrom logic, which checks that the caller has an allowance to burn tokens on behalf of the account.

Last updated