Unable to find manual contract audit (e.g. Certik, PeckShield, Solidity...)
Reentrancy in MYEScoin._transfer(address,address,uint256) (#1389-1445):
External calls:
- swapAndSendToFee(contractTokenBalance) (#1413)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1466-1472)
External calls sending eth:
- swapAndSendToFee(contractTokenBalance) (#1413)
- address(address(_marketingWalletAddress)).transfer(newbalance.mul(1).div(2)) (#1452)
- address(address(constructionWllet)).transfer(newbalance.mul(1).div(2)) (#1453)
State variables written after the call(s):
- super._transfer(from,address(this),fees) (#1430)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#833)
- _balances[recipient] = _balances[recipient].add(amount) (#834)
- super._transfer(from,to,amount) (#1433)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#833)
- _balances[recipient] = _balances[recipient].add(amount) (#834)
- swapping = false (#1414)
Reentrancy in DividendPayingToken._withdrawDividendOfUser(address) (#1027-1043):
External calls:
- (success) = user.call{gas: 3000,value: _withdrawableDividend}() (#1032)
State variables written after the call(s):
- withdrawnDividends[user] = withdrawnDividends[user].sub(_withdrawableDividend) (#1035)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in MYEScoin.updateDividendTracker(address) (#1234-1249):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (#1241)
- newDividendTracker.excludeFromDividends(address(this)) (#1242)
- newDividendTracker.excludeFromDividends(owner()) (#1243)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (#1244)
State variables written after the call(s):
- dividendTracker = newDividendTracker (#1248)
Apply the check-effects-interactions pattern.
Additional information: link
MYEScoin._transfer(address,address,uint256).iterations (#1441) is a local variable never initialized
MYEScoin._transfer(address,address,uint256).claims (#1441) is a local variable never initialized
MYEScoin._transfer(address,address,uint256).lastProcessedIndex (#1441) 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
MYEScoin.claim() (#1376-1378) ignores return value by dividendTracker.processAccount(msg.sender,false) (#1377)
MYEScoin._transfer(address,address,uint256) (#1389-1445) ignores return value by dividendTracker.process(gas) (#1441-1443)
Ensure that all the return values of the function calls are used.
Additional information: link
DividendPayingToken.constructor(string,string)._name (#997) shadows:
- ERC20._name (#655) (state variable)
DividendPayingToken.constructor(string,string)._symbol (#997) shadows:
- ERC20._symbol (#656) (state variable)
Rename the local variables that shadow another component.
Additional information: link
MYEScoin._setSwapTokenAmount(uint256) (#1226-1228) should emit an event for:
- swapTokensAtAmount = SwapTokenAmount (#1227)
Emit an event for critical parameter changes.
Additional information: link
MYEScoin.updateUniswapV2Router(address)._uniswapV2Pair (#1255-1256) lacks a zero-check on :
- uniswapV2Pair = _uniswapV2Pair (#1257)
MYEScoin.setMarketingWallet(address).wallet (#1275) lacks a zero-check on :
- _marketingWalletAddress = wallet (#1276)
MYEScoin.setConstructionWllet(address)._constructionWllet (#1278) lacks a zero-check on :
- constructionWllet = _constructionWllet (#1279)
Check that the address is not zero.
Additional information: link
DividendPayingToken._withdrawDividendOfUser(address) (#1027-1043) has external calls inside a loop: (success) = user.call{gas: 3000,value: _withdrawableDividend}() (#1032)
Favor pull over push strategy for external calls.
Additional information: link
Variable 'MYEScoin._transfer(address,address,uint256).claims (#1441)' in MYEScoin._transfer(address,address,uint256) (#1389-1445) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1442)
Variable 'MYEScoin._transfer(address,address,uint256).iterations (#1441)' in MYEScoin._transfer(address,address,uint256) (#1389-1445) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1442)
Variable 'MYEScoin._transfer(address,address,uint256).lastProcessedIndex (#1441)' in MYEScoin._transfer(address,address,uint256) (#1389-1445) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1442)
Move all variable declarations prior to any usage of the variable, and ensure that reaching a variable declaration does not depend on some conditional if it is used unconditionally.
Additional information: link
Reentrancy in MYEScoin.constructor() (#1190-1220):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1196-1197)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (#1200)
- uniswapV2Router = _uniswapV2Router (#1199)
Reentrancy in MYEScoin.constructor() (#1190-1220):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1196-1197)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1202)
- dividendTracker.excludeFromDividends(pair) (#1301)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (#1205)
- dividendTracker.excludeFromDividends(address(this)) (#1206)
- dividendTracker.excludeFromDividends(assign) (#1207)
- dividendTracker.excludeFromDividends(deadWallet) (#1208)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (#1209)
State variables written after the call(s):
- _mint(assign,100000000000000 * (10 ** 18)) (#1219)
- _balances[account] = _balances[account].add(amount) (#853)
- excludeFromFees(owner(),true) (#1212)
- _isExcludedFromFees[account] = excluded (#1262)
- excludeFromFees(assign,true) (#1213)
- _isExcludedFromFees[account] = excluded (#1262)
- excludeFromFees(_marketingWalletAddress,true) (#1214)
- _isExcludedFromFees[account] = excluded (#1262)
- excludeFromFees(constructionWllet,true) (#1215)
- _isExcludedFromFees[account] = excluded (#1262)
- excludeFromFees(address(this),true) (#1216)
- _isExcludedFromFees[account] = excluded (#1262)
- _mint(assign,100000000000000 * (10 ** 18)) (#1219)
- _totalSupply = _totalSupply.add(amount) (#852)
Reentrancy in BiHODLDividendTracker.processAccount(address,bool) (#1682-1692):
External calls:
- amount = _withdrawDividendOfUser(account) (#1683)
- (success) = user.call{gas: 3000,value: _withdrawableDividend}() (#1032)
State variables written after the call(s):
- lastClaimTimes[account] = block.timestamp (#1686)
Reentrancy in MYEScoin.updateUniswapV2Router(address) (#1251-1258):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(uniswapV2Router.factory()).createPair(address(this),uniswapV2Router.WETH()) (#1255-1256)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (#1257)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in MYEScoin._setAutomatedMarketMakerPair(address,bool) (#1296-1305):
External calls:
- dividendTracker.excludeFromDividends(pair) (#1301)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (#1304)
Reentrancy in MYEScoin._transfer(address,address,uint256) (#1389-1445):
External calls:
- swapAndSendToFee(contractTokenBalance) (#1413)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1466-1472)
External calls sending eth:
- swapAndSendToFee(contractTokenBalance) (#1413)
- address(address(_marketingWalletAddress)).transfer(newbalance.mul(1).div(2)) (#1452)
- address(address(constructionWllet)).transfer(newbalance.mul(1).div(2)) (#1453)
Event emitted after the call(s):
- Transfer(sender,recipient,amount) (#835)
- super._transfer(from,to,amount) (#1433)
- Transfer(sender,recipient,amount) (#835)
- super._transfer(from,address(this),fees) (#1430)
Reentrancy in MYEScoin._transfer(address,address,uint256) (#1389-1445):
External calls:
- swapAndSendToFee(contractTokenBalance) (#1413)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (#1466-1472)
- dividendTracker.setBalance(address(from),balanceOf(from)) (#1435)
- dividendTracker.setBalance(address(to),balanceOf(to)) (#1436)
- dividendTracker.process(gas) (#1441-1443)
External calls sending eth:
- swapAndSendToFee(contractTokenBalance) (#1413)
- address(address(_marketingWalletAddress)).transfer(newbalance.mul(1).div(2)) (#1452)
- address(address(constructionWllet)).transfer(newbalance.mul(1).div(2)) (#1453)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1442)
Reentrancy in MYEScoin.constructor() (#1190-1220):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1196-1197)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1202)
- dividendTracker.excludeFromDividends(pair) (#1301)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (#1304)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1202)
Reentrancy in MYEScoin.constructor() (#1190-1220):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#1196-1197)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (#1202)
- dividendTracker.excludeFromDividends(pair) (#1301)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (#1205)
- dividendTracker.excludeFromDividends(address(this)) (#1206)
- dividendTracker.excludeFromDividends(assign) (#1207)
- dividendTracker.excludeFromDividends(deadWallet) (#1208)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (#1209)
Event emitted after the call(s):
- ExcludeFromFees(account,excluded) (#1264)
- excludeFromFees(constructionWllet,true) (#1215)
- ExcludeFromFees(account,excluded) (#1264)
- excludeFromFees(_marketingWalletAddress,true) (#1214)
- ExcludeFromFees(account,excluded) (#1264)
- excludeFromFees(assign,true) (#1213)
- ExcludeFromFees(account,excluded) (#1264)
- excludeFromFees(owner(),true) (#1212)
- ExcludeFromFees(account,excluded) (#1264)
- excludeFromFees(address(this),true) (#1216)
- Transfer(address(0),account,amount) (#854)
- _mint(assign,100000000000000 * (10 ** 18)) (#1219)
Reentrancy in BiHODLDividendTracker.processAccount(address,bool) (#1682-1692):
External calls:
- amount = _withdrawDividendOfUser(account) (#1683)
- (success) = user.call{gas: 3000,value: _withdrawableDividend}() (#1032)
Event emitted after the call(s):
- Claim(account,amount,automatic) (#1687)
Reentrancy in MYEScoin.processDividendTracker(uint256) (#1371-1374):
External calls:
- (iterations,claims,lastProcessedIndex) = dividendTracker.process(gas) (#1372)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,false,gas,tx.origin) (#1373)
Reentrancy in MYEScoin.updateDividendTracker(address) (#1234-1249):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (#1241)
- newDividendTracker.excludeFromDividends(address(this)) (#1242)
- newDividendTracker.excludeFromDividends(owner()) (#1243)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (#1244)
Event emitted after the call(s):
- UpdateDividendTracker(newAddress,address(dividendTracker)) (#1246)
Apply the check-effects-interactions pattern.
Additional information: link
BiHODLDividendTracker.getAccount(address) (#1546-1589) uses timestamp for comparisons
Dangerous comparisons:
- nextClaimTime > block.timestamp (#1586-1588)
BiHODLDividendTracker.canAutoClaim(uint256) (#1610-1616) uses timestamp for comparisons
Dangerous comparisons:
- lastClaimTime > block.timestamp (#1611)
- block.timestamp.sub(lastClaimTime) >= claimWait (#1615)
Avoid relying on block.timestamp.
Additional information: link
Context._msgData() (#88-91) is never used and should be removed
DividendPayingToken._transfer(address,address,uint256) (#1083-1089) is never used and should be removed
SafeMath.mod(uint256,uint256) (#492-494) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#508-511) is never used and should be removed
SafeMathInt.abs(int256) (#563-566) is never used and should be removed
SafeMathInt.div(int256,int256) (#534-540) is never used and should be removed
SafeMathInt.mul(int256,int256) (#522-529) is never used and should be removed
Remove unused functions.
Additional information: link
MYEScoin.totalFees (#1146) is set pre-construction with a non-constant function or state variable:
- BNBRewardsFee.add(liquidityFee)
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 DividendPayingToken._withdrawDividendOfUser(address) (#1027-1043):
- (success) = user.call{gas: 3000,value: _withdrawableDividend}() (#1032)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence
Additional information: link
Function IUniswapV2Router01.WETH() (#97) is not in mixedCase
Function IUniswapV2Pair.DOMAIN_SEPARATOR() (#263) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (#264) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (#281) is not in mixedCase
Parameter DividendPayingToken.dividendOf(address)._owner (#1049) is not in mixedCase
Parameter DividendPayingToken.withdrawableDividendOf(address)._owner (#1056) is not in mixedCase
Parameter DividendPayingToken.withdrawnDividendOf(address)._owner (#1063) is not in mixedCase
Parameter DividendPayingToken.accumulativeDividendOf(address)._owner (#1073) is not in mixedCase
Constant DividendPayingToken.magnitude (#987) is not in UPPER_CASE_WITH_UNDERSCORES
Function MYEScoin._setSwapTokenAmount(uint256) (#1226-1228) is not in mixedCase
Parameter MYEScoin._setSwapTokenAmount(uint256).SwapTokenAmount (#1226) is not in mixedCase
Parameter MYEScoin.setConstructionWllet(address)._constructionWllet (#1278) is not in mixedCase
Parameter MYEScoin.setFixedSwitch(bool)._fixedSwitch (#1283) is not in mixedCase
Parameter MYEScoin.setSwappingSwitsh(bool)._swappingSwitsh (#1286) is not in mixedCase
Variable MYEScoin._marketingWalletAddress (#1138) is not in mixedCase
Variable MYEScoin.BNBRewardsFee (#1144) is not in mixedCase
Event BiHODLDividendTrackermintokenUpdated(uint256,uint256) (#1495) is not in CapWords
Parameter BiHODLDividendTracker.getAccount(address)._account (#1546) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#89)" inContext (#83-92)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Reentrancy in MYEScoin._transfer(address,address,uint256) (#1389-1445):
External calls:
- swapAndSendToFee(contractTokenBalance) (#1413)
- address(address(_marketingWalletAddress)).transfer(newbalance.mul(1).div(2)) (#1452)
- address(address(constructionWllet)).transfer(newbalance.mul(1).div(2)) (#1453)
State variables written after the call(s):
- super._transfer(from,address(this),fees) (#1430)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#833)
- _balances[recipient] = _balances[recipient].add(amount) (#834)
- super._transfer(from,to,amount) (#1433)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#833)
- _balances[recipient] = _balances[recipient].add(amount) (#834)
- swapping = false (#1414)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (#1442)
- Transfer(sender,recipient,amount) (#835)
- super._transfer(from,to,amount) (#1433)
- Transfer(sender,recipient,amount) (#835)
- super._transfer(from,address(this),fees) (#1430)
Apply the check-effects-interactions pattern.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#102) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#103)
Variable DividendPayingToken._withdrawDividendOfUser(address)._withdrawableDividend (#1028) is too similar to BiHODLDividendTracker.getAccount(address).withdrawableDividends (#1551)
Prevent variables from having similar names.
Additional information: link
MYEScoin.constructor() (#1190-1220) uses literals with too many digits:
- _mint(assign,100000000000000 * (10 ** 18)) (#1219)
MYEScoin.updateGasForProcessing(uint256) (#1308-1313) uses literals with too many digits:
- require(bool,string)(newValue >= 200000 && newValue <= 500000,BiHODL: gasForProcessing must be between 200,000 and 500,000) (#1309)
MYEScoin.slitherConstructorVariables() (#1127-1476) uses literals with too many digits:
- deadWallet = 0x000000000000000000000000000000000000dEaD (#1137)
MYEScoin.slitherConstructorVariables() (#1127-1476) uses literals with too many digits:
- swapTokensAtAmount = 300000000000 * (10 ** 18) (#1141)
MYEScoin.slitherConstructorVariables() (#1127-1476) uses literals with too many digits:
- gasForProcessing = 300000 (#1156)
BiHODLDividendTracker.constructor() (#1499-1502) uses literals with too many digits:
- minimumTokenBalanceForDividends = 100000000 * (10 ** 18) (#1501)
BiHODLDividendTracker.getAccountAtIndex(uint256) (#1591-1608) uses literals with too many digits:
- (0x0000000000000000000000000000000000000000,- 1,- 1,0,0,0,0,0) (#1602)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
SafeMathInt.MAX_INT256 (#517) is never used in SafeMathInt (#515-573)
Remove unused state variables.
Additional information: link
MYEScoin.BNBRewardsFee (#1144) should be constant
MYEScoin.deadWallet (#1137) should be constant
MYEScoin.liquidityFee (#1145) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#354-357)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#363-367)
get(IterableMapping.Map,address) should be declared external:
- IterableMapping.get(IterableMapping.Map,address) (#593-595)
getIndexOfKey(IterableMapping.Map,address) should be declared external:
- IterableMapping.getIndexOfKey(IterableMapping.Map,address) (#597-602)
getKeyAtIndex(IterableMapping.Map,uint256) should be declared external:
- IterableMapping.getKeyAtIndex(IterableMapping.Map,uint256) (#604-606)
size(IterableMapping.Map) should be declared external:
- IterableMapping.size(IterableMapping.Map) (#610-612)
name() should be declared external:
- ERC20.name() (#675-677)
symbol() should be declared external:
- ERC20.symbol() (#683-685)
decimals() should be declared external:
- ERC20.decimals() (#700-702)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (#726-729)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (#734-736)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (#745-748)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (#763-771)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (#785-788)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (#804-807)
withdrawDividend() should be declared external:
- BiHODLDividendTracker.withdrawDividend() (#1509-1511)
- DividendPayingToken.withdrawDividend() (#1021-1023)
dividendOf(address) should be declared external:
- DividendPayingToken.dividendOf(address) (#1049-1051)
withdrawnDividendOf(address) should be declared external:
- DividendPayingToken.withdrawnDividendOf(address) (#1063-1065)
updateDividendTracker(address) should be declared external:
- MYEScoin.updateDividendTracker(address) (#1234-1249)
updateUniswapV2Router(address) should be declared external:
- MYEScoin.updateUniswapV2Router(address) (#1251-1258)
excludeMultipleAccountsFromFees(address[],bool) should be declared external:
- MYEScoin.excludeMultipleAccountsFromFees(address[],bool) (#1267-1273)
setAutomatedMarketMakerPair(address,bool) should be declared external:
- MYEScoin.setAutomatedMarketMakerPair(address,bool) (#1289-1293)
updateGasForProcessing(uint256) should be declared external:
- MYEScoin.updateGasForProcessing(uint256) (#1308-1313)
isExcludedFromFees(address) should be declared external:
- MYEScoin.isExcludedFromFees(address) (#1327-1329)
withdrawableDividendOf(address) should be declared external:
- MYEScoin.withdrawableDividendOf(address) (#1331-1333)
dividendTokenBalanceOf(address) should be declared external:
- MYEScoin.dividendTokenBalanceOf(address) (#1335-1337)
getAccountAtIndex(uint256) should be declared external:
- BiHODLDividendTracker.getAccountAtIndex(uint256) (#1591-1608)
process(uint256) should be declared external:
- BiHODLDividendTracker.process(uint256) (#1635-1680)
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