Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
DogeHoly._splitOtherToken() (#1230-1276) ignores return value by doge.transfer(user,thisAmount.mul(rate).div(10000)) (#1258)
DogeHoly._splitOtherToken() (#1230-1276) ignores return value by doge.transfer(user,thisAmount.mul(rate).div(10000)) (#1269)
Use SafeERC20, or ensure that the transfer/transferFrom return value is checked.
Additional information: link
DogeHoly.startTime (#1033) is never initialized. It is used in:
- DogeHoly.isIn3minter() (#1116-1118)
Initialize all the variables. If a variable is meant to be initialized to zero, explicitly set it to zero to improve code readability.
Additional information: link
Function IUniswapV2Pair.DOMAIN_SEPARATOR() (#49) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#51) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#82) is not in mixedCase
Function IUniswapV2Router01.WETH() (#816) is not in mixedCase
Parameter DogeHoly.setSwapTokensAtAmount(uint256)._swapTokensAtAmount (#1100) is not in mixedCase
Parameter DogeHoly.setSwapAndLiquifyEnabled(bool)._enabled (#1104) is not in mixedCase
Function DogeHoly._splitOtherToken() (#1230-1276) is not in mixedCase
Variable DogeHoly._receive (#1027) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#10)" inContext (#4-14)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#821) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#822)
Prevent variables from having similar names.
Additional information: link
DogeHoly.slitherConstructorVariables() (#1019-1283) uses literals with too many digits:
- _destroyAddress = address(0x000000000000000000000000000000000000dEaD) (#1028)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
DogeHoly._transfer(address,address,uint256) (#1120-1169) performs a multiplication on the result of a division:
-super._transfer(from,address(this),amount.div(50).mul(3)) (#1157)
DogeHoly._transfer(address,address,uint256) (#1120-1169) performs a multiplication on the result of a division:
-amount = amount.div(10).mul(9) (#1159)
DogeHoly._splitOtherToken() (#1230-1276) performs a multiplication on the result of a division:
-rate = balanceOf(user).mul(10000).div(totalAmount) (#1267)
-doge.transfer(user,thisAmount.mul(rate).div(10000)) (#1269)
DogeHoly._splitOtherToken() (#1230-1276) performs a multiplication on the result of a division:
-doge.transfer(user,thisAmount.mul(rate).div(10000)) (#1258)
-rate = balanceOf(user).mul(10000).div(totalAmount) (#1267)
Consider ordering multiplication before division.
Additional information: link
Reentrancy in DogeHoly._transfer(address,address,uint256) (#1120-1169):
External calls:
- swapAndLiquifyV3(balanceOf(address(this))) (#1140)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1209-1215)
State variables written after the call(s):
- swapping = false (#1142)
Reentrancy in DogeHoly._transfer(address,address,uint256) (#1120-1169):
External calls:
- swapAndLiquifyV3(balanceOf(address(this))) (#1140)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1209-1215)
- _splitOtherToken() (#1146)
- doge.transfer(user,thisAmount.mul(rate).div(10000)) (#1258)
- doge.transfer(user,thisAmount.mul(rate).div(10000)) (#1269)
State variables written after the call(s):
- super._transfer(from,uniswapV2Pair,amount.div(50)) (#1153)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#558-561)
- _balances[recipient] = _balances[recipient].add(amount) (#562)
- super._transfer(from,_destroyAddress,amount.div(50)) (#1155)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#558-561)
- _balances[recipient] = _balances[recipient].add(amount) (#562)
- super._transfer(from,address(this),amount.div(50).mul(3)) (#1157)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#558-561)
- _balances[recipient] = _balances[recipient].add(amount) (#562)
- super._transfer(from,to,amount) (#1161)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#558-561)
- _balances[recipient] = _balances[recipient].add(amount) (#562)
- buyUser.push(to) (#1165)
Apply the check-effects-interactions pattern.
Additional information: link
DogeHoly._splitOtherToken().startIndex (#1236) is a local variable never initialized
Initialize all the variables. If a variable is meant to be initialized to zero, explicitly set it to zero to improve code readability.
Additional information: link
ERC20.allowance(address,address).owner (#414) shadows:
- Ownable.owner() (#277-279) (function)
ERC20._approve(address,address,uint256).owner (#625) shadows:
- Ownable.owner() (#277-279) (function)
Rename the local variables that shadow another component.
Additional information: link
DogeHoly.setSwapTokensAtAmount(uint256) (#1100-1102) should emit an event for:
- swapTokensAtAmount = _swapTokensAtAmount (#1101)
Emit an event for critical parameter changes.
Additional information: link
Ownable.constructor().msgSender (#269) lacks a zero-check on :
- _owner = msgSender (#270)
Check that the address is not zero.
Additional information: link
DogeHoly._splitOtherToken() (#1230-1276) has external calls inside a loop: doge.transfer(user,thisAmount.mul(rate).div(10000)) (#1258)
DogeHoly._splitOtherToken() (#1230-1276) has external calls inside a loop: doge.transfer(user,thisAmount.mul(rate).div(10000)) (#1269)
Favor pull over push strategy for external calls.
Additional information: link
Reentrancy in DogeHoly._transfer(address,address,uint256) (#1120-1169):
External calls:
- swapAndLiquifyV3(balanceOf(address(this))) (#1140)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1209-1215)
- _splitOtherToken() (#1146)
- doge.transfer(user,thisAmount.mul(rate).div(10000)) (#1258)
- doge.transfer(user,thisAmount.mul(rate).div(10000)) (#1269)
State variables written after the call(s):
- havePush[to] = true (#1164)
Reentrancy in DogeHoly.constructor(address) (#1055-1076):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1060-1061)
State variables written after the call(s):
- _approve(address(this),address(0x10ED43C718714eb63d5aA57B78B54704E256024E),10 ** 26) (#1062)
- _allowances[owner][spender] = amount (#632)
- _mint(tokenOwner,total) (#1075)
- _balances[account] = _balances[account].add(amount) (#581)
- excludeFromFees(tokenOwner,true) (#1069)
- _isExcludedFromFees[account] = excluded (#1088)
- excludeFromFees(address(this),true) (#1070)
- _isExcludedFromFees[account] = excluded (#1088)
- _mint(tokenOwner,total) (#1075)
- _totalSupply = _totalSupply.add(amount) (#580)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1066)
- automatedMarketMakerPairs[pair] = value (#1109)
- doge = IERC20(0xbA2aE424d960c26247Dd6c32edC70B295c744C43) (#1071)
- minBuyAmount = total.div(1000) (#1074)
- swapTokensAtAmount = total.div(10000) (#1073)
- uniswapV2Pair = _uniswapV2Pair (#1064)
- uniswapV2Router = _uniswapV2Router (#1063)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in DogeHoly._transfer(address,address,uint256) (#1120-1169):
External calls:
- swapAndLiquifyV3(balanceOf(address(this))) (#1140)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1209-1215)
- _splitOtherToken() (#1146)
- doge.transfer(user,thisAmount.mul(rate).div(10000)) (#1258)
- doge.transfer(user,thisAmount.mul(rate).div(10000)) (#1269)
Event emitted after the call(s):
- Transfer(sender,recipient,amount) (#563)
- super._transfer(from,_destroyAddress,amount.div(50)) (#1155)
- Transfer(sender,recipient,amount) (#563)
- super._transfer(from,uniswapV2Pair,amount.div(50)) (#1153)
- Transfer(sender,recipient,amount) (#563)
- super._transfer(from,to,amount) (#1161)
- Transfer(sender,recipient,amount) (#563)
- super._transfer(from,address(this),amount.div(50).mul(3)) (#1157)
Reentrancy in DogeHoly.constructor(address) (#1055-1076):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1060-1061)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#633)
- _approve(address(this),address(0x10ED43C718714eb63d5aA57B78B54704E256024E),10 ** 26) (#1062)
- ExcludeFromFees(account,excluded) (#1089)
- excludeFromFees(address(this),true) (#1070)
- ExcludeFromFees(account,excluded) (#1089)
- excludeFromFees(tokenOwner,true) (#1069)
- Transfer(address(0),account,amount) (#582)
- _mint(tokenOwner,total) (#1075)
Apply the check-effects-interactions pattern.
Additional information: link
DogeHoly.isIn3minter() (#1116-1118) uses timestamp for comparisons
Dangerous comparisons:
- startTime.add(180) > block.timestamp (#1117)
Avoid relying on block.timestamp.
Additional information: link
Context._msgData() (#9-13) is never used and should be removed
ERC20._burn(address,uint256) (#596-607) is never used and should be removed
SafeMath.sub(uint256,uint256) (#686-688) is never used and should be removed
Remove unused functions.
Additional information: link
DogeHoly._destroyAddress (#1028) should be constant
DogeHoly._receive (#1027) should be constant
DogeHoly.startTime (#1033) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
owner() should be declared external:
- Ownable.owner() (#277-279)
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#296-299)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#305-312)
name() should be declared external:
- ERC20.name() (#344-346)
symbol() should be declared external:
- ERC20.symbol() (#352-354)
decimals() should be declared external:
- ERC20.decimals() (#369-371)
totalSupply() should be declared external:
- ERC20.totalSupply() (#376-378)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (#401-409)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (#414-422)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (#431-439)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (#454-469)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (#483-494)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (#510-524)
updateUniswapV2Router(address) should be declared external:
- DogeHoly.updateUniswapV2Router(address) (#1082-1085)
excludeMultipleAccountsFromFees(address[],bool) should be declared external:
- DogeHoly.excludeMultipleAccountsFromFees(address[],bool) (#1092-1098)
setSwapTokensAtAmount(uint256) should be declared external:
- DogeHoly.setSwapTokensAtAmount(uint256) (#1100-1102)
setSwapAndLiquifyEnabled(bool) should be declared external:
- DogeHoly.setSwapAndLiquifyEnabled(bool) (#1104-1106)
isExcludedFromFees(address) should be declared external:
- DogeHoly.isExcludedFromFees(address) (#1112-1114)
isIn3minter() should be declared external:
- DogeHoly.isIn3minter() (#1116-1118)
swapAndLiquifyV1(uint256) should be declared external:
- DogeHoly.swapAndLiquifyV1(uint256) (#1171-1175)
swapAndLiquifyV4() should be declared external:
- DogeHoly.swapAndLiquifyV4() (#1198-1200)
rescueToken(address,uint256) should be declared external:
- DogeHoly.rescueToken(address,uint256) (#1220-1226)
getBuysize() should be declared external:
- DogeHoly.getBuysize() (#1278-1280)
Use the external attribute for functions never called from the contract.
Additional information: link
Unable to find website, listings and other project-related information
Young tokens have high risks of scam / price dump / death
Token has no active CoinGecko listing / rank
Token has no active CoinMarketCap listing / rank
Unable to find Telegram and Twitter accounts