CallistoSecurity / Smart-contract-auditing

This is a working repo of @EthereumCommonwealth audits. We performed more than 400 security audits since 2018. Not even a single contract was hacked after our auditors approved the code. Accepting audit requests here.
https://audits.callisto.network/
2 stars 2 forks source link

Binance-Peg Dogecoin Token (DOGE) #4

Closed yuriy77k closed 1 year ago

yuriy77k commented 1 year ago

Audit request

Binance-Peg Dogecoin Token (DOGE)

Source code

https://bscscan.com/address/0xba2ae424d960c26247dd6c32edc70b295c744c43

Payment plan

... Write [x] at the checkbox of the payment plan that suits your needs ...

Disclosure policy

Standard disclosure policy.

Contact information (optional)

Public

Platform

BSC

yuriy77k commented 1 year ago

Binance-Peg Dogecoin Token (DOGE) Security Audit Report

1. Summary

Binance-Peg Dogecoin Token (DOGE) smart contract security audit report performed by Callisto Security Audit Department

The Binance-Peg Dogecoin Token (DOGE) is an upgradable contract where the owner can change the executable contract whenever they want. In this report, we audit contract implementation https://bscscan.com/address/0xba5fe23f8a3a24bed3236f05f2fcf35fd0bf0b5c#code that was active on 30 April 2023

2. In scope

3. Findings

In total, 1 issue were reported, including:

In total, 2 notes were reported, including:

No critical security issues were found.

3.1. Known vulnerabilities of ERC-20 token

Severity: low

Description

Lack of transaction handling mechanism issue. WARNING! This is a very common issue, and it already caused millions of dollars in losses for lots of token users! More details here.

Recommendation

Add the following code to the transfer(_to address, ...) function:

require( _to != address(this) );

3.2. Owner privileges

Severity: owner privileges

Description

  1. Token uses an upgradable proxy contract where the owner can change the implementation contract whenever they want. A new implementation may have issues or dangerous functionality for users.
  2. Owner can mint any amount of tokens to themselves.

4. Security practices

5. Conclusion

The audited smart contract can be deployed. Only a low severity issue was found during the audit.

Users must pay attention to unlimited contract owner's rights.

It is recommended to adhere to the security practices described in pt. 4 of this report to ensure the contract's operability and prevent any issues that are not directly related to the code of this smart contract.