Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
Reentrancy in AliFist._transfer(address,address,uint256) (#1210-1286):
External calls:
- swapAndLiquify(contractTokenBalance) (#1273)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1320-1326)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1273)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#1285)
- _balances[_burn] = _balances[_burn].add(tBurn) (#1194)
- _balances[_market] = _balances[_market].add(tMarket) (#1186)
- _balances[address(this)] = _balances[address(this)].add(tLiquidity) (#1190)
- _balances[sender] = _balances[sender].sub(amount) (#1359)
- _balances[recipient] = _balances[recipient].add(tTransferAmount) (#1360)
Apply the check-effects-interactions pattern.
Additional information: link
Contract ticker (Ali Fist) contains non-alphanumeric characters.
Not a direct threat, but may indicate unreliable intentions of developer. Non-alphanumeric chars (,.;!#*&") are extremely rare among low risk tokens.
Reentrancy in AliFist._transfer(address,address,uint256) (#1210-1286):
External calls:
- swapAndLiquify(contractTokenBalance) (#1273)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1320-1326)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1273)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
State variables written after the call(s):
- _tokenTransfer(from,to,amount,takeFee) (#1285)
- _burnFee = _previousBurnFee (#1155)
- _burnFee = 0 (#1149)
- _tokenTransfer(from,to,amount,takeFee) (#1285)
- _liquidityFee = _previousLiquidityFee (#1156)
- _liquidityFee = 0 (#1150)
- _tokenTransfer(from,to,amount,takeFee) (#1285)
- _marketFee = _previousMarketFee (#1154)
- _marketFee = 0 (#1148)
- _tokenTransfer(from,to,amount,takeFee) (#1285)
- _previousBurnFee = _burnFee (#1145)
- _tokenTransfer(from,to,amount,takeFee) (#1285)
- _previousLiquidityFee = _liquidityFee (#1146)
- _tokenTransfer(from,to,amount,takeFee) (#1285)
- _previousMarketFee = _marketFee (#1144)
Reentrancy in AliFist.constructor() (#923-950):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#930-931)
State variables written after the call(s):
- _isExcludedFromFee[owner()] = true (#940)
- _isExcludedFromFee[address(this)] = true (#941)
- _isExcludedFromFee[_market] = true (#942)
- _isExcludedFromFee[_liquify] = true (#943)
- _liquify = owner() (#939)
- automatedMarketMakerPairs[uniswapV2Pair] = true (#936)
- uniswapV2Router = _uniswapV2Router (#934)
Reentrancy in AliFist.swapAndLiquify(uint256) (#1288-1309):
External calls:
- swapTokensForEth(half) (#1300)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1320-1326)
- addLiquidity(otherHalf,newBalance) (#1306)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1306)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (#1306)
- _allowances[owner][spender] = amount (#1206)
Reentrancy in AliFist.transferFrom(address,address,uint256) (#999-1014):
External calls:
- _transfer(sender,recipient,amount) (#1004)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1320-1326)
External calls sending eth:
- _transfer(sender,recipient,amount) (#1004)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
State variables written after the call(s):
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#1005-1012)
- _allowances[owner][spender] = amount (#1206)
Apply the check-effects-interactions pattern.
Additional information: link
AliFist.addLiquidity(uint256,uint256) (#1329-1342) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
Ensure that all the return values of the function calls are used.
Additional information: link
AliFist.allowance(address,address).owner (#981) shadows:
- Ownable.owner() (#465-467) (function)
AliFist._approve(address,address,uint256).owner (#1199) shadows:
- Ownable.owner() (#465-467) (function)
Rename the local variables that shadow another component.
Additional information: link
AliFist.setLiquidityFeePercent(uint256) (#1075-1078) should emit an event for:
- _liquidityFee = liquidityFee (#1076)
- _previousLiquidityFee = _liquidityFee (#1077)
AliFist.setMarketFeePercent(uint256) (#1080-1083) should emit an event for:
- _marketFee = marketFee (#1081)
- _previousMarketFee = _marketFee (#1082)
AliFist.setBurnFeePercent(uint256) (#1093-1096) should emit an event for:
- _burnFee = burnFee (#1094)
- _previousBurnFee = _burnFee (#1095)
AliFist.setMaxTxAmount(uint256) (#1098-1100) should emit an event for:
- _maxTxAmount = maxTxAmount (#1099)
AliFist.setMaxHavAmount(uint256) (#1102-1104) should emit an event for:
- _maxHavAmount = maxHavAmount (#1103)
AliFist.setNumTokensSellToAddToLiquidity(uint256) (#1111-1113) should emit an event for:
- numTokensSellToAddToLiquidity = num (#1112)
Emit an event for critical parameter changes.
Additional information: link
AliFist.setMarketAddress(address).market (#1085) lacks a zero-check on :
- _market = market (#1086)
AliFist.setLiquifyAddress(address).liquify (#1089) lacks a zero-check on :
- _liquify = liquify (#1090)
Check that the address is not zero.
Additional information: link
AliFist.swapTokensForEth(uint256) (#1311-1327) has external calls inside a loop: path[1] = uniswapV2Router.WETH() (#1315)
AliFist.swapTokensForEth(uint256) (#1311-1327) has external calls inside a loop: uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1320-1326)
AliFist.addLiquidity(uint256,uint256) (#1329-1342) has external calls inside a loop: uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
Favor pull over push strategy for external calls.
Additional information: link
Reentrancy in AliFist._transfer(address,address,uint256) (#1210-1286):
External calls:
- swapAndLiquify(contractTokenBalance) (#1273)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1320-1326)
External calls sending eth:
- swapAndLiquify(contractTokenBalance) (#1273)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
Event emitted after the call(s):
- Transfer(sender,_burn,tBurn) (#1195)
- _tokenTransfer(from,to,amount,takeFee) (#1285)
- Transfer(sender,recipient,tTransferAmount) (#1364)
- _tokenTransfer(from,to,amount,takeFee) (#1285)
Reentrancy in AliFist.constructor() (#923-950):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#930-931)
Event emitted after the call(s):
- Transfer(address(0x4EFf6e0c168F14baDEfDF3aa8593b9aE61d02F8f),_msgSender(),_tTotal) (#945-949)
Reentrancy in AliFist.swapAndLiquify(uint256) (#1288-1309):
External calls:
- swapTokensForEth(half) (#1300)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1320-1326)
- addLiquidity(otherHalf,newBalance) (#1306)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (#1306)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1207)
- addLiquidity(otherHalf,newBalance) (#1306)
- SwapAndLiquify(half,newBalance,otherHalf) (#1308)
Reentrancy in AliFist.transferFrom(address,address,uint256) (#999-1014):
External calls:
- _transfer(sender,recipient,amount) (#1004)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1320-1326)
External calls sending eth:
- _transfer(sender,recipient,amount) (#1004)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (#1334-1341)
Event emitted after the call(s):
- Approval(owner,spender,amount) (#1207)
- _approve(sender,_msgSender(),_allowances[sender][_msgSender()].sub(amount,ERC20: transfer amount exceeds allowance)) (#1005-1012)
Apply the check-effects-interactions pattern.
Additional information: link
Ownable.unlock() (#515-523) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp > _lockTime,Contract is locked until 7 days) (#520)
AliFist.setStartTimeForSwap() (#1115-1119) uses timestamp for comparisons
Dangerous comparisons:
- _startTimeForSwap == 0 (#1116)
AliFist._transfer(address,address,uint256) (#1210-1286) uses timestamp for comparisons
Dangerous comparisons:
- _startTimeForSwap == 0 && ! _isExcludedFromFee[from] && ! _isExcludedFromFee[to] (#1239-1241)
Avoid relying on block.timestamp.
Additional information: link
Address.isContract(address) (#274-285) uses assembly
- INLINE ASM (#281-283)
Address._functionCallWithValue(address,bytes,uint256,string) (#400-428) uses assembly
- INLINE ASM (#420-423)
Do not use evm assembly.
Additional information: link
AliFist.lockTheSwap() (#917-921) has costly operations inside a loop:
- inSwapAndLiquify = true (#918)
AliFist.lockTheSwap() (#917-921) has costly operations inside a loop:
- inSwapAndLiquify = false (#920)
AliFist.removeAllFee() (#1141-1151) has costly operations inside a loop:
- _previousMarketFee = _marketFee (#1144)
AliFist.removeAllFee() (#1141-1151) has costly operations inside a loop:
- _previousBurnFee = _burnFee (#1145)
AliFist.removeAllFee() (#1141-1151) has costly operations inside a loop:
- _previousLiquidityFee = _liquidityFee (#1146)
AliFist.removeAllFee() (#1141-1151) has costly operations inside a loop:
- _marketFee = 0 (#1148)
AliFist.removeAllFee() (#1141-1151) has costly operations inside a loop:
- _burnFee = 0 (#1149)
AliFist.removeAllFee() (#1141-1151) has costly operations inside a loop:
- _liquidityFee = 0 (#1150)
AliFist.restoreAllFee() (#1153-1157) has costly operations inside a loop:
- _marketFee = _previousMarketFee (#1154)
AliFist.restoreAllFee() (#1153-1157) has costly operations inside a loop:
- _burnFee = _previousBurnFee (#1155)
AliFist.restoreAllFee() (#1153-1157) has costly operations inside a loop:
- _liquidityFee = _previousLiquidityFee (#1156)
Use a local variable to hold the loop computation result.
Additional information: link
Address._functionCallWithValue(address,bytes,uint256,string) (#400-428) is never used and should be removed
Address.functionCall(address,bytes) (#335-340) is never used and should be removed
Address.functionCall(address,bytes,string) (#348-354) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (#367-379) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (#387-398) is never used and should be removed
Address.isContract(address) (#274-285) is never used and should be removed
Address.sendValue(address,uint256) (#303-315) is never used and should be removed
Context._msgData() (#247-250) is never used and should be removed
SafeMath.mod(uint256,uint256) (#216-218) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#232-239) is never used and should be removed
Remove unused functions.
Additional information: link
AliFist._previousMarketFee (#886) is set pre-construction with a non-constant function or state variable:
- _marketFee
AliFist._previousLiquidityFee (#889) is set pre-construction with a non-constant function or state variable:
- _liquidityFee
AliFist._previousBurnFee (#892) is set pre-construction with a non-constant function or state variable:
- _burnFee
Remove any initialization of state variables via non-constant state variables or function calls. If variables must be set upon contract deployment, locate initialization in the constructor instead.
Additional information: link
Low level call in Address.sendValue(address,uint256) (#303-315):
- (success) = recipient.call{value: amount}() (#310)
Low level call in Address._functionCallWithValue(address,bytes,uint256,string) (#400-428):
- (success,returndata) = target.call{value: weiValue}(data) (#409-411)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence
Additional information: link
Function IUniswapV2Pair.DOMAIN_SEPARATOR() (#589) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#591) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#622) is not in mixedCase
Function IUniswapV2Router.WETH() (#668) is not in mixedCase
Parameter AliFist.setSwapAndLiquifyEnabled(bool)._enabled (#1106) is not in mixedCase
Parameter AliFist.calculateLiquidityFee(uint256)._amount (#1121) is not in mixedCase
Parameter AliFist.calculateMarketFee(uint256)._amount (#1129) is not in mixedCase
Parameter AliFist.calculateBurnFee(uint256)._amount (#1137) is not in mixedCase
Function AliFist._getValues(uint256) (#1166-1183) is not in mixedCase
Variable AliFist._marketFee (#885) is not in mixedCase
Variable AliFist._liquidityFee (#888) is not in mixedCase
Variable AliFist._burnFee (#891) is not in mixedCase
Variable AliFist._startTimeForSwap (#894) is not in mixedCase
Variable AliFist._maxTxAmount (#902) is not in mixedCase
Variable AliFist._maxHavAmount (#903) is not in mixedCase
Variable AliFist._burn (#906) is not in mixedCase
Variable AliFist._market (#907) is not in mixedCase
Variable AliFist._liquify (#908) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#248)" inContext (#242-251)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Variable IUniswapV2Router.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#673) is too similar to IUniswapV2Router.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#674)
Prevent variables from having similar names.
Additional information: link
AliFist.slitherConstructorVariables() (#869-1369) uses literals with too many digits:
- _burn = 0x000000000000000000000000000000000000dEaD (#906)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
AliFist.MAX (#878) is never used in AliFist (#869-1369)
Remove unused state variables.
Additional information: link
AliFist._burn (#906) should be constant
AliFist._decimals (#883) should be constant
AliFist._name (#881) should be constant
AliFist._symbol (#882) should be constant
AliFist._tTotal (#879) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#484-487)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#493-500)
geUnlockTime() should be declared external:
- Ownable.geUnlockTime() (#502-504)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (#507-512)
unlock() should be declared external:
- Ownable.unlock() (#515-523)
name() should be declared external:
- AliFist.name() (#952-954)
symbol() should be declared external:
- AliFist.symbol() (#956-958)
decimals() should be declared external:
- AliFist.decimals() (#960-962)
totalSupply() should be declared external:
- AliFist.totalSupply() (#964-966)
transfer(address,uint256) should be declared external:
- AliFist.transfer(address,uint256) (#972-979)
allowance(address,address) should be declared external:
- AliFist.allowance(address,address) (#981-988)
approve(address,uint256) should be declared external:
- AliFist.approve(address,uint256) (#990-997)
transferFrom(address,address,uint256) should be declared external:
- AliFist.transferFrom(address,address,uint256) (#999-1014)
increaseAllowance(address,uint256) should be declared external:
- AliFist.increaseAllowance(address,uint256) (#1016-1027)
decreaseAllowance(address,uint256) should be declared external:
- AliFist.decreaseAllowance(address,uint256) (#1029-1043)
excludeFromFee(address) should be declared external:
- AliFist.excludeFromFee(address) (#1045-1047)
includeInFee(address) should be declared external:
- AliFist.includeInFee(address) (#1049-1051)
excludeMultiFromFee(address[],bool) should be declared external:
- AliFist.excludeMultiFromFee(address[],bool) (#1053-1060)
setMarketAddress(address) should be declared external:
- AliFist.setMarketAddress(address) (#1085-1087)
setLiquifyAddress(address) should be declared external:
- AliFist.setLiquifyAddress(address) (#1089-1091)
setSwapAndLiquifyEnabled(bool) should be declared external:
- AliFist.setSwapAndLiquifyEnabled(bool) (#1106-1109)
setNumTokensSellToAddToLiquidity(uint256) should be declared external:
- AliFist.setNumTokensSellToAddToLiquidity(uint256) (#1111-1113)
isExcludedFromFee(address) should be declared external:
- AliFist.isExcludedFromFee(address) (#1159-1161)
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