Next Generation Of Gamified DeFi & Metaverse
DeFiDegenLand.addLiquidity(uint256,uint256) (DeFiDegenLand.sol#435-450) sends eth to arbitrary user
Dangerous calls:
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
Ensure that an arbitrary user cannot withdraw unauthorized funds.
Additional information: link
Reentrancy in DeFiDegenLand._transfer(address,address,uint256) (DeFiDegenLand.sol#289-360):
External calls:
- swapAndSendToFee(marketingTokens) (DeFiDegenLand.sol#316)
- IERC20(BTC).transfer(_marketingWalletAddress,newBalance) (DeFiDegenLand.sol#368)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#426-432)
- swapAndLiquify(swapTokens) (DeFiDegenLand.sol#319)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#406-412)
- swapAndSendDividends(sellTokens) (DeFiDegenLand.sol#322)
- success = IERC20(BTC).transfer(address(dividendTracker),dividends) (DeFiDegenLand.sol#455)
- dividendTracker.distributeBTCDividends(dividends) (DeFiDegenLand.sol#458)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#426-432)
External calls sending eth:
- swapAndLiquify(swapTokens) (DeFiDegenLand.sol#319)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
State variables written after the call(s):
- super._transfer(from,address(this),fees) (DeFiDegenLand.sol#342)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (ERC20.sol#221)
- _balances[recipient] = _balances[recipient].add(amount) (ERC20.sol#222)
- super._transfer(from,to,amount) (DeFiDegenLand.sol#345)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (ERC20.sol#221)
- _balances[recipient] = _balances[recipient].add(amount) (ERC20.sol#222)
- swapping = false (DeFiDegenLand.sol#324)
Apply the check-effects-interactions pattern.
Additional information: link
DeFiDegenLand.swapAndSendToFee(uint256) (DeFiDegenLand.sol#362-369) ignores return value by IERC20(BTC).transfer(_marketingWalletAddress,newBalance) (DeFiDegenLand.sol#368)
Use SafeERC20, or ensure that the transfer/transferFrom return value is checked.
Additional information: link
Combination 2: Unchecked transfer + Functions that send Ether to arbitraty destination. Usual for scams. May be justified by some complex mechanics (e.g. rebase, reflections). DYOR & manual audit are advised.
Combination 3: Reentrancy vulnerabilities + Unchecked transfer vulnerability. Usual for scams. May be justified by some complex mechanics (e.g. rebase, reflections). DYOR & manual audit are advised.
Combination 1: Reentrancy vulnerabilities + Functions that send Ether to arbitraty destination. Usual for scams. May be justified by some complex mechanics (e.g. rebase, reflections). DYOR & manual audit are advised.
DeFiDegenLand.constructor() (DeFiDegenLand.sol#87-119) uses literals with too many digits:
- _mint(owner(),10000000000000 * (10 ** 18)) (DeFiDegenLand.sol#118)
DeFiDegenLand.updateGasForProcessing(uint256) (DeFiDegenLand.sol#210-215) uses literals with too many digits:
- require(bool,string)(newValue >= 200000 && newValue <= 500000,DeFiDegenLand: gasForProcessing must be between 200,000 and 500,000) (DeFiDegenLand.sol#211)
DeFiDegenLand.slitherConstructorVariables() (DeFiDegenLand.sol#17-462) uses literals with too many digits:
- deadWallet = 0x000000000000000000000000000000000000dEaD (DeFiDegenLand.sol#27)
DeFiDegenLand.slitherConstructorVariables() (DeFiDegenLand.sol#17-462) uses literals with too many digits:
- swapTokensAtAmount = 2000000 * (10 ** 18) (DeFiDegenLand.sol#31)
DeFiDegenLand.slitherConstructorVariables() (DeFiDegenLand.sol#17-462) uses literals with too many digits:
- gasForProcessing = 300000 (DeFiDegenLand.sol#44)
DeFiDegenLandDividendTracker.getAccountAtIndex(uint256) (DeFiDegenLand.sol#569-586) uses literals with too many digits:
- (0x0000000000000000000000000000000000000000,- 1,- 1,0,0,0,0,0) (DeFiDegenLand.sol#580)
Use: Ether suffix, Time suffix, or The scientific notation
Additional information: link
SafeMathInt.MAX_INT256 (SafeMathInt.sol#36) is never used in SafeMathInt (SafeMathInt.sol#34-92)
Remove unused state variables.
Additional information: link
DeFiDegenLand.deadWallet (DeFiDegenLand.sol#27) should be constant
DeFiDegenLand.swapTokensAtAmount (DeFiDegenLand.sol#31) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
updateDividendTracker(address) should be declared external:
- DeFiDegenLand.updateDividendTracker(address) (DeFiDegenLand.sol#125-140)
updateUniswapV2Router(address) should be declared external:
- DeFiDegenLand.updateUniswapV2Router(address) (DeFiDegenLand.sol#142-149)
excludeMultipleAccountsFromFees(address[],bool) should be declared external:
- DeFiDegenLand.excludeMultipleAccountsFromFees(address[],bool) (DeFiDegenLand.sol#158-164)
setAutomatedMarketMakerPair(address,bool) should be declared external:
- DeFiDegenLand.setAutomatedMarketMakerPair(address,bool) (DeFiDegenLand.sol#187-191)
updateGasForProcessing(uint256) should be declared external:
- DeFiDegenLand.updateGasForProcessing(uint256) (DeFiDegenLand.sol#210-215)
isExcludedFromFees(address) should be declared external:
- DeFiDegenLand.isExcludedFromFees(address) (DeFiDegenLand.sol#229-231)
withdrawableDividendOf(address) should be declared external:
- DeFiDegenLand.withdrawableDividendOf(address) (DeFiDegenLand.sol#233-235)
dividendTokenBalanceOf(address) should be declared external:
- DeFiDegenLand.dividendTokenBalanceOf(address) (DeFiDegenLand.sol#237-239)
withdrawDividend() should be declared external:
- DeFiDegenLandDividendTracker.withdrawDividend() (DeFiDegenLand.sol#493-495)
- DividendPayingToken.withdrawDividend() (DividendPayingToken.sol#70-72)
getAccountAtIndex(uint256) should be declared external:
- DeFiDegenLandDividendTracker.getAccountAtIndex(uint256) (DeFiDegenLand.sol#569-586)
process(uint256) should be declared external:
- DeFiDegenLandDividendTracker.process(uint256) (DeFiDegenLand.sol#613-658)
distributeBTCDividends(uint256) should be declared external:
- DividendPayingToken.distributeBTCDividends(uint256) (DividendPayingToken.sol#55-66)
dividendOf(address) should be declared external:
- DividendPayingToken.dividendOf(address) (DividendPayingToken.sol#98-100)
withdrawnDividendOf(address) should be declared external:
- DividendPayingToken.withdrawnDividendOf(address) (DividendPayingToken.sol#112-114)
name() should be declared external:
- ERC20.name() (ERC20.sol#63-65)
symbol() should be declared external:
- ERC20.symbol() (ERC20.sol#71-73)
decimals() should be declared external:
- ERC20.decimals() (ERC20.sol#88-90)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (ERC20.sol#114-117)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (ERC20.sol#122-124)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (ERC20.sol#133-136)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (ERC20.sol#151-159)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (ERC20.sol#173-176)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (ERC20.sol#192-195)
get(IterableMapping.Map,address) should be declared external:
- IterableMapping.get(IterableMapping.Map,address) (IterableMapping.sol#13-15)
getIndexOfKey(IterableMapping.Map,address) should be declared external:
- IterableMapping.getIndexOfKey(IterableMapping.Map,address) (IterableMapping.sol#17-22)
getKeyAtIndex(IterableMapping.Map,uint256) should be declared external:
- IterableMapping.getKeyAtIndex(IterableMapping.Map,uint256) (IterableMapping.sol#24-26)
size(IterableMapping.Map) should be declared external:
- IterableMapping.size(IterableMapping.Map) (IterableMapping.sol#30-32)
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (Ownable.sol#43-46)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (Ownable.sol#52-56)
Use the external attribute for functions never called from the contract.
Additional information: link
DividendPayingToken._withdrawDividendOfUser(address) (DividendPayingToken.sol#76-92) has external calls inside a loop: success = IERC20(BTC).transfer(user,_withdrawableDividend) (DividendPayingToken.sol#81)
Favor pull over push strategy for external calls.
Additional information: link
Reentrancy in DividendPayingToken._withdrawDividendOfUser(address) (DividendPayingToken.sol#76-92):
External calls:
- success = IERC20(BTC).transfer(user,_withdrawableDividend) (DividendPayingToken.sol#81)
State variables written after the call(s):
- withdrawnDividends[user] = withdrawnDividends[user].sub(_withdrawableDividend) (DividendPayingToken.sol#84)
Reentrancy in DeFiDegenLand.updateDividendTracker(address) (DeFiDegenLand.sol#125-140):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (DeFiDegenLand.sol#132)
- newDividendTracker.excludeFromDividends(address(this)) (DeFiDegenLand.sol#133)
- newDividendTracker.excludeFromDividends(owner()) (DeFiDegenLand.sol#134)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (DeFiDegenLand.sol#135)
State variables written after the call(s):
- dividendTracker = newDividendTracker (DeFiDegenLand.sol#139)
Apply the check-effects-interactions pattern.
Additional information: link
DeFiDegenLand._transfer(address,address,uint256).claims (DeFiDegenLand.sol#353) is a local variable never initialized
DeFiDegenLand._transfer(address,address,uint256).lastProcessedIndex (DeFiDegenLand.sol#353) is a local variable never initialized
DeFiDegenLand._transfer(address,address,uint256).iterations (DeFiDegenLand.sol#353) 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
DeFiDegenLand.claim() (DeFiDegenLand.sol#276-278) ignores return value by dividendTracker.processAccount(msg.sender,false) (DeFiDegenLand.sol#277)
DeFiDegenLand._transfer(address,address,uint256) (DeFiDegenLand.sol#289-360) ignores return value by dividendTracker.process(gas) (DeFiDegenLand.sol#353-358)
DeFiDegenLand.addLiquidity(uint256,uint256) (DeFiDegenLand.sol#435-450) ignores return value by uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
Ensure that all the return values of the function calls are used.
Additional information: link
DividendPayingToken.constructor(string,string)._name (DividendPayingToken.sol#50) shadows:
- ERC20._name (ERC20.sol#43) (state variable)
DividendPayingToken.constructor(string,string)._symbol (DividendPayingToken.sol#50) shadows:
- ERC20._symbol (ERC20.sol#44) (state variable)
DividendPayingToken.dividendOf(address)._owner (DividendPayingToken.sol#98) shadows:
- Ownable._owner (Ownable.sol#8) (state variable)
DividendPayingToken.withdrawableDividendOf(address)._owner (DividendPayingToken.sol#105) shadows:
- Ownable._owner (Ownable.sol#8) (state variable)
DividendPayingToken.withdrawnDividendOf(address)._owner (DividendPayingToken.sol#112) shadows:
- Ownable._owner (Ownable.sol#8) (state variable)
DividendPayingToken.accumulativeDividendOf(address)._owner (DividendPayingToken.sol#122) shadows:
- Ownable._owner (Ownable.sol#8) (state variable)
Rename the local variables that shadow another component.
Additional information: link
DeFiDegenLand.setBTCRewardsFee(uint256) (DeFiDegenLand.sol#170-173) should emit an event for:
- BTCRewardsFee = value (DeFiDegenLand.sol#171)
- totalFees = BTCRewardsFee.add(liquidityFee).add(marketingFee) (DeFiDegenLand.sol#172)
DeFiDegenLand.setLiquiditFee(uint256) (DeFiDegenLand.sol#175-178) should emit an event for:
- liquidityFee = value (DeFiDegenLand.sol#176)
- totalFees = BTCRewardsFee.add(liquidityFee).add(marketingFee) (DeFiDegenLand.sol#177)
DeFiDegenLand.setMarketingFee(uint256) (DeFiDegenLand.sol#180-184) should emit an event for:
- marketingFee = value (DeFiDegenLand.sol#181)
- totalFees = BTCRewardsFee.add(liquidityFee).add(marketingFee) (DeFiDegenLand.sol#182)
Emit an event for critical parameter changes.
Additional information: link
DeFiDegenLand.updateUniswapV2Router(address)._uniswapV2Pair (DeFiDegenLand.sol#146-147) lacks a zero-check on :
- uniswapV2Pair = _uniswapV2Pair (DeFiDegenLand.sol#148)
DeFiDegenLand.setMarketingWallet(address).wallet (DeFiDegenLand.sol#166) lacks a zero-check on :
- _marketingWalletAddress = wallet (DeFiDegenLand.sol#167)
Check that the address is not zero.
Additional information: link
Variable 'DeFiDegenLand._transfer(address,address,uint256).iterations (DeFiDegenLand.sol#353)' in DeFiDegenLand._transfer(address,address,uint256) (DeFiDegenLand.sol#289-360) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (DeFiDegenLand.sol#354)
Variable 'DeFiDegenLand._transfer(address,address,uint256).lastProcessedIndex (DeFiDegenLand.sol#353)' in DeFiDegenLand._transfer(address,address,uint256) (DeFiDegenLand.sol#289-360) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (DeFiDegenLand.sol#354)
Variable 'DeFiDegenLand._transfer(address,address,uint256).claims (DeFiDegenLand.sol#353)' in DeFiDegenLand._transfer(address,address,uint256) (DeFiDegenLand.sol#289-360) potentially used before declaration: ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (DeFiDegenLand.sol#354)
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 DeFiDegenLand._transfer(address,address,uint256) (DeFiDegenLand.sol#289-360):
External calls:
- swapAndSendToFee(marketingTokens) (DeFiDegenLand.sol#316)
- IERC20(BTC).transfer(_marketingWalletAddress,newBalance) (DeFiDegenLand.sol#368)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#426-432)
- swapAndLiquify(swapTokens) (DeFiDegenLand.sol#319)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#406-412)
External calls sending eth:
- swapAndLiquify(swapTokens) (DeFiDegenLand.sol#319)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
State variables written after the call(s):
- swapAndLiquify(swapTokens) (DeFiDegenLand.sol#319)
- _allowances[owner][spender] = amount (ERC20.sol#287)
Reentrancy in DeFiDegenLand._transfer(address,address,uint256) (DeFiDegenLand.sol#289-360):
External calls:
- swapAndSendToFee(marketingTokens) (DeFiDegenLand.sol#316)
- IERC20(BTC).transfer(_marketingWalletAddress,newBalance) (DeFiDegenLand.sol#368)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#426-432)
- swapAndLiquify(swapTokens) (DeFiDegenLand.sol#319)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#406-412)
- swapAndSendDividends(sellTokens) (DeFiDegenLand.sol#322)
- success = IERC20(BTC).transfer(address(dividendTracker),dividends) (DeFiDegenLand.sol#455)
- dividendTracker.distributeBTCDividends(dividends) (DeFiDegenLand.sol#458)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#426-432)
External calls sending eth:
- swapAndLiquify(swapTokens) (DeFiDegenLand.sol#319)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
State variables written after the call(s):
- swapAndSendDividends(sellTokens) (DeFiDegenLand.sol#322)
- _allowances[owner][spender] = amount (ERC20.sol#287)
Reentrancy in DeFiDegenLand.constructor() (DeFiDegenLand.sol#87-119):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (DeFiDegenLand.sol#94-95)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (DeFiDegenLand.sol#98)
- uniswapV2Router = _uniswapV2Router (DeFiDegenLand.sol#97)
Reentrancy in DeFiDegenLand.constructor() (DeFiDegenLand.sol#87-119):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (DeFiDegenLand.sol#94-95)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (DeFiDegenLand.sol#100)
- dividendTracker.excludeFromDividends(pair) (DeFiDegenLand.sol#203)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (DeFiDegenLand.sol#103)
- dividendTracker.excludeFromDividends(address(this)) (DeFiDegenLand.sol#104)
- dividendTracker.excludeFromDividends(owner()) (DeFiDegenLand.sol#105)
- dividendTracker.excludeFromDividends(deadWallet) (DeFiDegenLand.sol#106)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (DeFiDegenLand.sol#107)
State variables written after the call(s):
- _mint(owner(),10000000000000 * (10 ** 18)) (DeFiDegenLand.sol#118)
- _balances[account] = _balances[account].add(amount) (ERC20.sol#241)
- excludeFromFees(owner(),true) (DeFiDegenLand.sol#110)
- _isExcludedFromFees[account] = excluded (DeFiDegenLand.sol#153)
- excludeFromFees(_marketingWalletAddress,true) (DeFiDegenLand.sol#111)
- _isExcludedFromFees[account] = excluded (DeFiDegenLand.sol#153)
- excludeFromFees(address(this),true) (DeFiDegenLand.sol#112)
- _isExcludedFromFees[account] = excluded (DeFiDegenLand.sol#153)
- _mint(owner(),10000000000000 * (10 ** 18)) (DeFiDegenLand.sol#118)
- _totalSupply = _totalSupply.add(amount) (ERC20.sol#240)
Reentrancy in DeFiDegenLandDividendTracker.processAccount(address,bool) (DeFiDegenLand.sol#660-670):
External calls:
- amount = _withdrawDividendOfUser(account) (DeFiDegenLand.sol#661)
- success = IERC20(BTC).transfer(user,_withdrawableDividend) (DividendPayingToken.sol#81)
State variables written after the call(s):
- lastClaimTimes[account] = block.timestamp (DeFiDegenLand.sol#664)
Reentrancy in DeFiDegenLand.swapAndLiquify(uint256) (DeFiDegenLand.sol#371-392):
External calls:
- swapTokensForEth(half) (DeFiDegenLand.sol#383)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#406-412)
- addLiquidity(otherHalf,newBalance) (DeFiDegenLand.sol#389)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (DeFiDegenLand.sol#389)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
State variables written after the call(s):
- addLiquidity(otherHalf,newBalance) (DeFiDegenLand.sol#389)
- _allowances[owner][spender] = amount (ERC20.sol#287)
Reentrancy in DeFiDegenLand.updateUniswapV2Router(address) (DeFiDegenLand.sol#142-149):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(uniswapV2Router.factory()).createPair(address(this),uniswapV2Router.WETH()) (DeFiDegenLand.sol#146-147)
State variables written after the call(s):
- uniswapV2Pair = _uniswapV2Pair (DeFiDegenLand.sol#148)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in DeFiDegenLand._setAutomatedMarketMakerPair(address,bool) (DeFiDegenLand.sol#198-207):
External calls:
- dividendTracker.excludeFromDividends(pair) (DeFiDegenLand.sol#203)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (DeFiDegenLand.sol#206)
Reentrancy in DeFiDegenLand._transfer(address,address,uint256) (DeFiDegenLand.sol#289-360):
External calls:
- swapAndSendToFee(marketingTokens) (DeFiDegenLand.sol#316)
- IERC20(BTC).transfer(_marketingWalletAddress,newBalance) (DeFiDegenLand.sol#368)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#426-432)
- swapAndLiquify(swapTokens) (DeFiDegenLand.sol#319)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#406-412)
External calls sending eth:
- swapAndLiquify(swapTokens) (DeFiDegenLand.sol#319)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- swapAndLiquify(swapTokens) (DeFiDegenLand.sol#319)
- SwapAndLiquify(half,newBalance,otherHalf) (DeFiDegenLand.sol#391)
- swapAndLiquify(swapTokens) (DeFiDegenLand.sol#319)
Reentrancy in DeFiDegenLand._transfer(address,address,uint256) (DeFiDegenLand.sol#289-360):
External calls:
- swapAndSendToFee(marketingTokens) (DeFiDegenLand.sol#316)
- IERC20(BTC).transfer(_marketingWalletAddress,newBalance) (DeFiDegenLand.sol#368)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#426-432)
- swapAndLiquify(swapTokens) (DeFiDegenLand.sol#319)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#406-412)
- swapAndSendDividends(sellTokens) (DeFiDegenLand.sol#322)
- success = IERC20(BTC).transfer(address(dividendTracker),dividends) (DeFiDegenLand.sol#455)
- dividendTracker.distributeBTCDividends(dividends) (DeFiDegenLand.sol#458)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#426-432)
External calls sending eth:
- swapAndLiquify(swapTokens) (DeFiDegenLand.sol#319)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- swapAndSendDividends(sellTokens) (DeFiDegenLand.sol#322)
- SendDividends(tokens,dividends) (DeFiDegenLand.sol#459)
- swapAndSendDividends(sellTokens) (DeFiDegenLand.sol#322)
- Transfer(sender,recipient,amount) (ERC20.sol#223)
- super._transfer(from,to,amount) (DeFiDegenLand.sol#345)
- Transfer(sender,recipient,amount) (ERC20.sol#223)
- super._transfer(from,address(this),fees) (DeFiDegenLand.sol#342)
Reentrancy in DeFiDegenLand._transfer(address,address,uint256) (DeFiDegenLand.sol#289-360):
External calls:
- swapAndSendToFee(marketingTokens) (DeFiDegenLand.sol#316)
- IERC20(BTC).transfer(_marketingWalletAddress,newBalance) (DeFiDegenLand.sol#368)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#426-432)
- swapAndLiquify(swapTokens) (DeFiDegenLand.sol#319)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#406-412)
- swapAndSendDividends(sellTokens) (DeFiDegenLand.sol#322)
- success = IERC20(BTC).transfer(address(dividendTracker),dividends) (DeFiDegenLand.sol#455)
- dividendTracker.distributeBTCDividends(dividends) (DeFiDegenLand.sol#458)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#426-432)
- dividendTracker.setBalance(address(from),balanceOf(from)) (DeFiDegenLand.sol#347)
- dividendTracker.setBalance(address(to),balanceOf(to)) (DeFiDegenLand.sol#348)
- dividendTracker.process(gas) (DeFiDegenLand.sol#353-358)
External calls sending eth:
- swapAndLiquify(swapTokens) (DeFiDegenLand.sol#319)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,true,gas,tx.origin) (DeFiDegenLand.sol#354)
Reentrancy in DeFiDegenLand.constructor() (DeFiDegenLand.sol#87-119):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (DeFiDegenLand.sol#94-95)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (DeFiDegenLand.sol#100)
- dividendTracker.excludeFromDividends(pair) (DeFiDegenLand.sol#203)
Event emitted after the call(s):
- SetAutomatedMarketMakerPair(pair,value) (DeFiDegenLand.sol#206)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (DeFiDegenLand.sol#100)
Reentrancy in DeFiDegenLand.constructor() (DeFiDegenLand.sol#87-119):
External calls:
- _uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (DeFiDegenLand.sol#94-95)
- _setAutomatedMarketMakerPair(_uniswapV2Pair,true) (DeFiDegenLand.sol#100)
- dividendTracker.excludeFromDividends(pair) (DeFiDegenLand.sol#203)
- dividendTracker.excludeFromDividends(address(dividendTracker)) (DeFiDegenLand.sol#103)
- dividendTracker.excludeFromDividends(address(this)) (DeFiDegenLand.sol#104)
- dividendTracker.excludeFromDividends(owner()) (DeFiDegenLand.sol#105)
- dividendTracker.excludeFromDividends(deadWallet) (DeFiDegenLand.sol#106)
- dividendTracker.excludeFromDividends(address(_uniswapV2Router)) (DeFiDegenLand.sol#107)
Event emitted after the call(s):
- ExcludeFromFees(account,excluded) (DeFiDegenLand.sol#155)
- excludeFromFees(address(this),true) (DeFiDegenLand.sol#112)
- ExcludeFromFees(account,excluded) (DeFiDegenLand.sol#155)
- excludeFromFees(_marketingWalletAddress,true) (DeFiDegenLand.sol#111)
- ExcludeFromFees(account,excluded) (DeFiDegenLand.sol#155)
- excludeFromFees(owner(),true) (DeFiDegenLand.sol#110)
- Transfer(address(0),account,amount) (ERC20.sol#242)
- _mint(owner(),10000000000000 * (10 ** 18)) (DeFiDegenLand.sol#118)
Reentrancy in DeFiDegenLandDividendTracker.processAccount(address,bool) (DeFiDegenLand.sol#660-670):
External calls:
- amount = _withdrawDividendOfUser(account) (DeFiDegenLand.sol#661)
- success = IERC20(BTC).transfer(user,_withdrawableDividend) (DividendPayingToken.sol#81)
Event emitted after the call(s):
- Claim(account,amount,automatic) (DeFiDegenLand.sol#665)
Reentrancy in DeFiDegenLand.processDividendTracker(uint256) (DeFiDegenLand.sol#271-274):
External calls:
- (iterations,claims,lastProcessedIndex) = dividendTracker.process(gas) (DeFiDegenLand.sol#272)
Event emitted after the call(s):
- ProcessedDividendTracker(iterations,claims,lastProcessedIndex,false,gas,tx.origin) (DeFiDegenLand.sol#273)
Reentrancy in DeFiDegenLand.swapAndLiquify(uint256) (DeFiDegenLand.sol#371-392):
External calls:
- swapTokensForEth(half) (DeFiDegenLand.sol#383)
- uniswapV2Router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#406-412)
- addLiquidity(otherHalf,newBalance) (DeFiDegenLand.sol#389)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
External calls sending eth:
- addLiquidity(otherHalf,newBalance) (DeFiDegenLand.sol#389)
- uniswapV2Router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,address(0),block.timestamp) (DeFiDegenLand.sol#441-448)
Event emitted after the call(s):
- Approval(owner,spender,amount) (ERC20.sol#288)
- addLiquidity(otherHalf,newBalance) (DeFiDegenLand.sol#389)
- SwapAndLiquify(half,newBalance,otherHalf) (DeFiDegenLand.sol#391)
Reentrancy in DeFiDegenLand.swapAndSendDividends(uint256) (DeFiDegenLand.sol#452-461):
External calls:
- swapTokensForBTC(tokens) (DeFiDegenLand.sol#453)
- uniswapV2Router.swapExactTokensForTokensSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (DeFiDegenLand.sol#426-432)
- success = IERC20(BTC).transfer(address(dividendTracker),dividends) (DeFiDegenLand.sol#455)
- dividendTracker.distributeBTCDividends(dividends) (DeFiDegenLand.sol#458)
Event emitted after the call(s):
- SendDividends(tokens,dividends) (DeFiDegenLand.sol#459)
Reentrancy in DeFiDegenLand.updateDividendTracker(address) (DeFiDegenLand.sol#125-140):
External calls:
- newDividendTracker.excludeFromDividends(address(newDividendTracker)) (DeFiDegenLand.sol#132)
- newDividendTracker.excludeFromDividends(address(this)) (DeFiDegenLand.sol#133)
- newDividendTracker.excludeFromDividends(owner()) (DeFiDegenLand.sol#134)
- newDividendTracker.excludeFromDividends(address(uniswapV2Router)) (DeFiDegenLand.sol#135)
Event emitted after the call(s):
- UpdateDividendTracker(newAddress,address(dividendTracker)) (DeFiDegenLand.sol#137)
Apply the check-effects-interactions pattern.
Additional information: link
DeFiDegenLandDividendTracker.getAccount(address) (DeFiDegenLand.sol#524-567) uses timestamp for comparisons
Dangerous comparisons:
- nextClaimTime > block.timestamp (DeFiDegenLand.sol#564-566)
DeFiDegenLandDividendTracker.canAutoClaim(uint256) (DeFiDegenLand.sol#588-594) uses timestamp for comparisons
Dangerous comparisons:
- lastClaimTime > block.timestamp (DeFiDegenLand.sol#589)
- block.timestamp.sub(lastClaimTime) >= claimWait (DeFiDegenLand.sol#593)
Avoid relying on block.timestamp.
Additional information: link
Context._msgData() (Context.sol#20-23) is never used and should be removed
DividendPayingToken._transfer(address,address,uint256) (DividendPayingToken.sol#132-138) is never used and should be removed
SafeMath.mod(uint256,uint256) (SafeMath.sol#126-128) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (SafeMath.sol#142-145) is never used and should be removed
SafeMathInt.abs(int256) (SafeMathInt.sol#82-85) is never used and should be removed
SafeMathInt.div(int256,int256) (SafeMathInt.sol#53-59) is never used and should be removed
SafeMathInt.mul(int256,int256) (SafeMathInt.sol#41-48) is never used and should be removed
Remove unused functions.
Additional information: link
DeFiDegenLand.totalFees (DeFiDegenLand.sol#38) is set pre-construction with a non-constant function or state variable:
- BTCRewardsFee.add(liquidityFee).add(marketingFee)
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
Pragma version^0.6.2 (Context.sol#3) allows old versions
Pragma version^0.6.2 (DeFiDegenLand.sol#6) allows old versions
Pragma version^0.6.2 (DividendPayingToken.sol#3) allows old versions
Pragma version^0.6.2 (DividendPayingTokenInterface.sol#3) allows old versions
Pragma version^0.6.2 (DividendPayingTokenOptionalInterface.sol#3) allows old versions
Pragma version^0.6.2 (ERC20.sol#3) allows old versions
Pragma version^0.6.2 (IERC20.sol#3) allows old versions
Pragma version^0.6.2 (IERC20Metadata.sol#3) allows old versions
Pragma version^0.6.2 (IUniswapV2Factory.sol#3) allows old versions
Pragma version^0.6.2 (IUniswapV2Pair.sol#3) allows old versions
Pragma version^0.6.2 (IUniswapV2Router.sol#3) allows old versions
Pragma version^0.6.2 (IterableMapping.sol#2) allows old versions
Pragma version^0.6.2 (Ownable.sol#1) allows old versions
Pragma version^0.6.2 (SafeMath.sol#3) allows old versions
Pragma version^0.6.2 (SafeMathInt.sol#28) allows old versions
Pragma version^0.6.2 (SafeMathUint.sol#3) allows old versions
Deploy with any of the following Solidity versions: 0.5.16 - 0.5.17, 0.6.11 - 0.6.12, 0.7.5 - 0.7.6 Use a simple pragma version that allows any of these versions. Consider using the latest version of Solidity for testing.
Additional information: link
Variable DeFiDegenLand.BTC (DeFiDegenLand.sol#29) is not in mixedCase
Variable DeFiDegenLand._isBlacklisted (DeFiDegenLand.sol#33) is not in mixedCase
Variable DeFiDegenLand.BTCRewardsFee (DeFiDegenLand.sol#35) is not in mixedCase
Variable DeFiDegenLand._marketingWalletAddress (DeFiDegenLand.sol#40) is not in mixedCase
Parameter DeFiDegenLandDividendTracker.getAccount(address)._account (DeFiDegenLand.sol#524) is not in mixedCase
Parameter DividendPayingToken.dividendOf(address)._owner (DividendPayingToken.sol#98) is not in mixedCase
Parameter DividendPayingToken.withdrawableDividendOf(address)._owner (DividendPayingToken.sol#105) is not in mixedCase
Parameter DividendPayingToken.withdrawnDividendOf(address)._owner (DividendPayingToken.sol#112) is not in mixedCase
Parameter DividendPayingToken.accumulativeDividendOf(address)._owner (DividendPayingToken.sol#122) is not in mixedCase
Variable DividendPayingToken.BTC (DividendPayingToken.sol#24) is not in mixedCase
Constant DividendPayingToken.magnitude (DividendPayingToken.sol#30) is not in UPPER_CASE_WITH_UNDERSCORES
Function IUniswapV2Pair.DOMAIN_SEPARATOR() (IUniswapV2Pair.sol#20) is not in mixedCase
Function IUniswapV2Pair.PERMIT_TYPEHASH() (IUniswapV2Pair.sol#21) is not in mixedCase
Function IUniswapV2Pair.MINIMUM_LIQUIDITY() (IUniswapV2Pair.sol#38) is not in mixedCase
Function IUniswapV2Router01.WETH() (IUniswapV2Router.sol#7) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (Context.sol#21)" inContext (Context.sol#15-24)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Variable DividendPayingToken._withdrawDividendOfUser(address)._withdrawableDividend (DividendPayingToken.sol#77) is too similar to DeFiDegenLandDividendTracker.getAccount(address).withdrawableDividends (DeFiDegenLand.sol#529)
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (IUniswapV2Router.sol#12) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (IUniswapV2Router.sol#13)
Prevent variables from having similar names.
Additional information: link
Unable to find token on CoinHunt
Additional information: link
Unable to find code repository for the project
Token is marked as scam (rug pull, honeypot, phishing, etc.)
Additional information: link
Young tokens have high risks of price dump / death
Young tokens have high risks of scam / price dump / death
Young tokens have high risks of price dump / death
Young tokens have high risks of price dump / death
Young tokens have high risks of price dump / death
Token has relatively low CoinGecko rank
Last post in Twitter was more than 30 days ago
Unable to find Youtube account
Unable to find Discord account