DKS [DarkShield] Token

Contract ownership is not renounced (belongs to a wallet)

Unable to verify that contract auditor is trusted: Certik, Quantstamp, Hacken, Solidity, Paladinsec, Openzeppelin, Verichains

Redundant expression "this (#51)" inContext (#43-54)
Remove redundant statements if they congest code but offer no value.

Additional information: link

Reentrancy in DarkShield._transfer(address,address,uint256) (#305-323):
External calls:
- BP.protect(sender,recipient,amount) (#311)
State variables written after the call(s):
- _balances[sender] = _balances[sender].sub(amount,BEP20: transfer amount exceeds balance) (#317-320)
- _balances[recipient] = _balances[recipient].add(amount) (#321)
Reentrancy in DarkShield.transferFrom(address,address,uint256) (#256-271):
External calls:
- _transfer(sender,recipient,amount) (#261)
- BP.protect(sender,recipient,amount) (#311)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,BEP20: transfer amount exceeds allowance)) (#262-269)
- _allowances[owner][spender] = amount (#344)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in DarkShield.transferFrom(address,address,uint256) (#256-271):
External calls:
- _transfer(sender,recipient,amount) (#261)
- BP.protect(sender,recipient,amount) (#311)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#345)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,BEP20: transfer amount exceeds allowance)) (#262-269)
Reentrancy in DarkShield._transfer(address,address,uint256) (#305-323):
External calls:
- BP.protect(sender,recipient,amount) (#311)
Event emitted after the call(s):
- Transfer(sender,recipient,amount) (#322)
Apply the check-effects-interactions pattern.

Additional information: link

DarkShield.setBotProtectionDisableForever() (#204-207) compares to a boolean constant:
-require(bool)(BPDisabledForever == false) (#205)
Remove the equality to the boolean constant.

Additional information: link

SafeMath.mul(uint256,uint256) (#79-88) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#109-116) is never used and should be removed
SafeMath.div(uint256,uint256,string) (#94-103) is never used and should be removed
Context._msgData() (#50-53) is never used and should be removed
SafeMath.mod(uint256,uint256) (#105-107) is never used and should be removed
SafeMath.div(uint256,uint256) (#90-92) is never used and should be removed
DarkShield._burnFrom(address,uint256) (#348-358) is never used and should be removed
Remove unused functions.

Additional information: link

Variable DarkShield.BPDisabledForever (#183) is not in mixedCase
Parameter DarkShield.setBPAddrss(address)._bp (#195) is not in mixedCase
Variable DarkShield.BP (#181) is not in mixedCase
Parameter DarkShield.setBpEnabled(bool)._enabled (#200) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

DarkShield.constructor() (#185-193) uses literals with too many digits:
- _totalSupply = 5000000000000000000000000000 (#189)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

decreaseAllowance(address,uint256) should be declared external:
- DarkShield.decreaseAllowance(address,uint256) (#285-298)
burn(uint256) should be declared external:
- DarkShield.burn(uint256) (#300-303)
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#142-145)
increaseAllowance(address,uint256) should be declared external:
- DarkShield.increaseAllowance(address,uint256) (#273-283)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#147-149)
Use the external attribute for functions never called from the contract.

Additional information: link

DarkShield._approve(address,address,uint256).owner (#337) shadows:
- Ownable.owner() (#133-135) (function)
DarkShield.allowance(address,address).owner (#242) shadows:
- Ownable.owner() (#133-135) (function)
Rename the local variables that shadow another component.

Additional information: link

Token is deployed only at one blockchain

Unable to find audit link on the website

Unable to find token on CoinHunt

Additional information: link

Unable to find code repository for the project

Young tokens have high risks of price dump / death

Token has relatively low CoinGecko rank