TAPE [ToolApe] Token

Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)

HasForeignAsset.getAsset(IBEP20) (#520-522) ignores return value by asset.transfer(owner(),this.assetBalance(asset)) (#521)
Use SafeERC20, or ensure that the transfer/transferFrom return value is checked.

Additional information: link

Contract locking ether found:
Contract ToolApe (#592-940) has payable functions:
- ToolApe.receive() (#618-620)
But does not have a function to withdraw the ether
Remove the payable attribute or add a withdraw function.

Additional information: link

ToolApe.allowance(address,address).owner (#690) shadows:
- Ownable.owner() (#489-491) (function)
ToolApe._approve(address,address,uint256).owner (#898) shadows:
- Ownable.owner() (#489-491) (function)
Rename the local variables that shadow another component.

Additional information: link

Pausable.pause() (#553-557) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp < 1621969429,Function is not callable anymore) (#554)
Avoid relying on block.timestamp.

Additional information: link

Address.isContract(address) (#194-205) uses assembly
- INLINE ASM (#201-203)
Address._functionCallWithValue(address,bytes,uint256,string) (#320-347) uses assembly
- INLINE ASM (#339-342)
Do not use evm assembly.

Additional information: link

Address._functionCallWithValue(address,bytes,uint256,string) (#320-347) is never used and should be removed
Address.functionCall(address,bytes) (#255-260) is never used and should be removed
Address.functionCall(address,bytes,string) (#268-274) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#287-299) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#307-318) is never used and should be removed
Address.isContract(address) (#194-205) is never used and should be removed
Address.sendValue(address,uint256) (#223-235) is never used and should be removed
Context._msgData() (#452-455) is never used and should be removed
SafeMath.div(uint256,uint256) (#107-109) is never used and should be removed
SafeMath.div(uint256,uint256,string) (#123-133) is never used and should be removed
SafeMath.mod(uint256,uint256) (#147-149) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#163-170) is never used and should be removed
ToolApe._setupDecimals(uint8) (#916-918) is never used and should be removed
Remove unused functions.

Additional information: link

Low level call in Address.sendValue(address,uint256) (#223-235):
- (success) = recipient.call{value: amount}() (#230)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#320-347):
- (success,returndata) = target.call{value: weiValue}(data) (#329-330)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence

Additional information: link

Redundant expression "this (#453)" inContext (#447-456)
Remove redundant statements if they congest code but offer no value.

Additional information: link

ToolApe.constructor() (#610-616) uses literals with too many digits:
- _maxSupply = 100000000 (#614)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#505-512)
pause() should be declared external:
- Pausable.pause() (#553-557)
unpause() should be declared external:
- Pausable.unpause() (#562-565)
name() should be declared external:
- ToolApe.name() (#625-627)
symbol() should be declared external:
- ToolApe.symbol() (#633-635)
decimals() should be declared external:
- ToolApe.decimals() (#650-652)
totalSupply() should be declared external:
- ToolApe.totalSupply() (#657-659)
balanceOf(address) should be declared external:
- ToolApe.balanceOf(address) (#664-666)
transfer(address,uint256) should be declared external:
- ToolApe.transfer(address,uint256) (#676-685)
allowance(address,address) should be declared external:
- ToolApe.allowance(address,address) (#690-698)
approve(address,uint256) should be declared external:
- ToolApe.approve(address,uint256) (#707-715)
transferFrom(address,address,uint256) should be declared external:
- ToolApe.transferFrom(address,address,uint256) (#729-744)
increaseAllowance(address,uint256) should be declared external:
- ToolApe.increaseAllowance(address,uint256) (#758-769)
decreaseAllowance(address,uint256) should be declared external:
- ToolApe.decreaseAllowance(address,uint256) (#785-799)
burn(uint256) should be declared external:
- ToolApe.burn(uint256) (#880-882)
Use the external attribute for functions never called from the contract.

Additional information: link