GWSPP [GoldenWspp] Token

Liquifier._addLiquidity(uint256,uint256) (goldenwspp.sol#890-919) sends eth to arbitrary user
Dangerous calls:
- (tokenAmountSent,ethAmountSent,liquidity) = _router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (goldenwspp.sol#895-907)
Liquifier.withdrawLockedEth(address) (goldenwspp.sol#951-959) sends eth to arbitrary user
Dangerous calls:
- recipient.transfer(amount) (goldenwspp.sol#958)
Ensure that an arbitrary user cannot withdraw unauthorized funds.

Additional information: link

Tokenomics._getFeeStruct(uint256) (goldenwspp.sol#242-245) contains a tautology or contradiction:
- require(bool,string)(index >= 0 && index < fees.length,FeesSettings._getFeeStruct: Fee index out of bounds) (goldenwspp.sol#243)
Fix the incorrect comparison by changing the value type or the comparison.

Additional information: link

BaseRfiToken.includeInReward(address) (goldenwspp.sol#501-512) has costly operations inside a loop:
- _excluded.pop() (goldenwspp.sol#508)
Use a local variable to hold the loop computation result.

Additional information: link

BaseRfiToken._getValues(uint256,uint256) (goldenwspp.sol#657-667) performs a multiplication on the result of a division:
-tTotalFees = tAmount.mul(feesSum).div(FEES_DIVISOR) (goldenwspp.sol#659)
-rTotalFees = tTotalFees.mul(currentRate) (goldenwspp.sol#663)
BaseRfiToken._redistribute(uint256,uint256,uint256,uint256) (goldenwspp.sol#721-727) performs a multiplication on the result of a division:
-tFee = amount.mul(fee).div(FEES_DIVISOR) (goldenwspp.sol#722)
-rFee = tFee.mul(currentRate) (goldenwspp.sol#723)
SafeToken._burn(uint256,uint256,uint256,uint256) (goldenwspp.sol#1097-1103) performs a multiplication on the result of a division:
-tBurn = amount.mul(fee).div(FEES_DIVISOR) (goldenwspp.sol#1098)
-rBurn = tBurn.mul(currentRate) (goldenwspp.sol#1099)
SafeToken._takeFee(uint256,uint256,uint256,address,uint256) (goldenwspp.sol#1105-1115) performs a multiplication on the result of a division:
-tAmount = amount.mul(fee).div(FEES_DIVISOR) (goldenwspp.sol#1107)
-rAmount = tAmount.mul(currentRate) (goldenwspp.sol#1108)
Consider ordering multiplication before division.

Additional information: link

BaseRfiToken._transfer(address,address,uint256) (goldenwspp.sol#540-593) uses tx.origin for authorization: require(bool,string)(block.number > _getLastBlock(tx.origin),You cant send more than one transaction in one block.) (goldenwspp.sol#575)
BaseRfiToken._transfer(address,address,uint256) (goldenwspp.sol#540-593) uses tx.origin for authorization: _getLastTransaction(tx.origin) > 0 (goldenwspp.sol#579)
BaseRfiToken._transfer(address,address,uint256) (goldenwspp.sol#540-593) uses tx.origin for authorization: require(bool,string)(_getLastTransaction(tx.origin) <= block.timestamp,You need to wait TIME_LIMIT_TRANSACTION before perform another transaction.) (goldenwspp.sol#581)
Do not use tx.origin for authorization.

Additional information: link

BaseRfiToken.allowance(address,address).owner (goldenwspp.sol#319) shadows:
- Ownable.owner() (safetoken-imports.sol#87-89) (function)
BaseRfiToken.getAddressLastTransaction(address).owner (goldenwspp.sol#328) shadows:
- Ownable.owner() (safetoken-imports.sol#87-89) (function)
BaseRfiToken._approve(address,address,uint256).owner (goldenwspp.sol#517) shadows:
- Ownable.owner() (safetoken-imports.sol#87-89) (function)
BaseRfiToken._transferTokens(address,address,uint256,bool).sumOfFees (goldenwspp.sol#605) shadows:
- Tokenomics.sumOfFees (goldenwspp.sol#209) (state variable)
BaseRfiToken._takeFees(uint256,uint256,uint256,bool,uint256).sumOfFees (goldenwspp.sol#651) shadows:
- Tokenomics.sumOfFees (goldenwspp.sol#209) (state variable)
Liquifier._approveDelegate(address,address,uint256).owner (goldenwspp.sol#965) shadows:
- Ownable.owner() (safetoken-imports.sol#87-89) (function)
SafeToken.constructor(Liquifier.Env)._env (goldenwspp.sol#1018) shadows:
- Liquifier._env (goldenwspp.sol#767) (state variable)
SafeToken._takeTransactionFees(uint256,uint256,bool,uint256).name (goldenwspp.sol#1072) shadows:
- BaseRfiToken.name() (goldenwspp.sol#299) (function)
- IERC20Metadata.name() (safetoken-imports.sol#17) (function)
SafeToken._approveDelegate(address,address,uint256).owner (goldenwspp.sol#1127) shadows:
- Ownable.owner() (safetoken-imports.sol#87-89) (function)
Rename the local variables that shadow another component.

Additional information: link

Manageable.transferManagement(address).newManager (safetoken-imports.sol#132) lacks a zero-check on :
- _manager = newManager (safetoken-imports.sol#134)
Check that the address is not zero.

Additional information: link

Reentrancy in Liquifier._addLiquidity(uint256,uint256) (goldenwspp.sol#890-919):
External calls:
- (tokenAmountSent,ethAmountSent,liquidity) = _router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (goldenwspp.sol#895-907)
State variables written after the call(s):
- withdrawableBalance = address(this).balance (goldenwspp.sol#917)
Reentrancy in Liquifier._setRouterAddress(address) (goldenwspp.sol#838-843):
External calls:
- _pair = IPancakeV2Factory(_newPancakeRouter.factory()).createPair(address(this),_newPancakeRouter.WETH()) (goldenwspp.sol#840)
State variables written after the call(s):
- _router = _newPancakeRouter (goldenwspp.sol#841)
Reentrancy in Liquifier._swapAndLiquify(uint256) (goldenwspp.sol#845-867):
External calls:
- _swapTokensForEth(half) (goldenwspp.sol#858)
- _router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (goldenwspp.sol#879-887)
- _addLiquidity(otherHalf,newBalance) (goldenwspp.sol#864)
- (tokenAmountSent,ethAmountSent,liquidity) = _router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (goldenwspp.sol#895-907)
External calls sending eth:
- _addLiquidity(otherHalf,newBalance) (goldenwspp.sol#864)
- (tokenAmountSent,ethAmountSent,liquidity) = _router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (goldenwspp.sol#895-907)
State variables written after the call(s):
- _addLiquidity(otherHalf,newBalance) (goldenwspp.sol#864)
- withdrawableBalance = address(this).balance (goldenwspp.sol#917)
Reentrancy in SafeToken.constructor(Liquifier.Env) (goldenwspp.sol#1018-1026):
External calls:
- initializeLiquiditySwapper(_env,maxTransactionAmount,numberOfTokensToSwapToLiquidity) (goldenwspp.sol#1020)
- _pair = IPancakeV2Factory(_newPancakeRouter.factory()).createPair(address(this),_newPancakeRouter.WETH()) (goldenwspp.sol#840)
State variables written after the call(s):
- _exclude(_pair) (goldenwspp.sol#1024)
- _balances[account] = tokenFromReflection(_reflectedBalances[account]) (goldenwspp.sol#495)
- _exclude(burnAddress) (goldenwspp.sol#1025)
- _balances[account] = tokenFromReflection(_reflectedBalances[account]) (goldenwspp.sol#495)
- _exclude(_pair) (goldenwspp.sol#1024)
- _excluded.push(account) (goldenwspp.sol#498)
- _exclude(burnAddress) (goldenwspp.sol#1025)
- _excluded.push(account) (goldenwspp.sol#498)
- _exclude(_pair) (goldenwspp.sol#1024)
- _isExcludedFromRewards[account] = true (goldenwspp.sol#497)
- _exclude(burnAddress) (goldenwspp.sol#1025)
- _isExcludedFromRewards[account] = true (goldenwspp.sol#497)
Reentrancy in SafeTokenV1Beta.constructor() (goldenwspp.sol#1134-1137):
External calls:
- SafeToken(Env.MainnetV2) (goldenwspp.sol#1134)
- _pair = IPancakeV2Factory(_newPancakeRouter.factory()).createPair(address(this),_newPancakeRouter.WETH()) (goldenwspp.sol#840)
State variables written after the call(s):
- _approve(owner(),address(_router),~ uint256(0)) (goldenwspp.sol#1136)
- _allowances[owner][spender] = amount (goldenwspp.sol#521)
Reentrancy in Liquifier.initializeLiquiditySwapper(Liquifier.Env,uint256,uint256) (goldenwspp.sol#800-809):
External calls:
- _setRouterAddress(_mainnetRouterV1Address) (goldenwspp.sol#802)
- _pair = IPancakeV2Factory(_newPancakeRouter.factory()).createPair(address(this),_newPancakeRouter.WETH()) (goldenwspp.sol#840)
- _setRouterAddress(_mainnetRouterV2Address) (goldenwspp.sol#803)
- _pair = IPancakeV2Factory(_newPancakeRouter.factory()).createPair(address(this),_newPancakeRouter.WETH()) (goldenwspp.sol#840)
- _setRouterAddress(_testnetRouterAddress) (goldenwspp.sol#804)
- _pair = IPancakeV2Factory(_newPancakeRouter.factory()).createPair(address(this),_newPancakeRouter.WETH()) (goldenwspp.sol#840)
State variables written after the call(s):
- maxTransactionAmount = maxTx (goldenwspp.sol#806)
- numberOfTokensToSwapToLiquidity = liquifyAmount (goldenwspp.sol#807)
Apply the check-effects-interactions pattern.

Additional information: link

Reentrancy in Liquifier._addLiquidity(uint256,uint256) (goldenwspp.sol#890-919):
External calls:
- (tokenAmountSent,ethAmountSent,liquidity) = _router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (goldenwspp.sol#895-907)
Event emitted after the call(s):
- LiquidityAdded(tokenAmountSent,ethAmountSent,liquidity) (goldenwspp.sol#918)
Reentrancy in Liquifier._setRouterAddress(address) (goldenwspp.sol#838-843):
External calls:
- _pair = IPancakeV2Factory(_newPancakeRouter.factory()).createPair(address(this),_newPancakeRouter.WETH()) (goldenwspp.sol#840)
Event emitted after the call(s):
- RouterSet(router) (goldenwspp.sol#842)
Reentrancy in Liquifier._swapAndLiquify(uint256) (goldenwspp.sol#845-867):
External calls:
- _swapTokensForEth(half) (goldenwspp.sol#858)
- _router.swapExactTokensForETHSupportingFeeOnTransferTokens(tokenAmount,0,path,address(this),block.timestamp) (goldenwspp.sol#879-887)
- _addLiquidity(otherHalf,newBalance) (goldenwspp.sol#864)
- (tokenAmountSent,ethAmountSent,liquidity) = _router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (goldenwspp.sol#895-907)
External calls sending eth:
- _addLiquidity(otherHalf,newBalance) (goldenwspp.sol#864)
- (tokenAmountSent,ethAmountSent,liquidity) = _router.addLiquidityETH{value: ethAmount}(address(this),tokenAmount,0,0,owner(),block.timestamp) (goldenwspp.sol#895-907)
Event emitted after the call(s):
- LiquidityAdded(tokenAmountSent,ethAmountSent,liquidity) (goldenwspp.sol#918)
- _addLiquidity(otherHalf,newBalance) (goldenwspp.sol#864)
- SwapAndLiquify(half,newBalance,otherHalf) (goldenwspp.sol#866)
Reentrancy in SafeTokenV1Beta.constructor() (goldenwspp.sol#1134-1137):
External calls:
- SafeToken(Env.MainnetV2) (goldenwspp.sol#1134)
- _pair = IPancakeV2Factory(_newPancakeRouter.factory()).createPair(address(this),_newPancakeRouter.WETH()) (goldenwspp.sol#840)
Event emitted after the call(s):
- Approval(owner,spender,amount) (goldenwspp.sol#522)
- _approve(owner(),address(_router),~ uint256(0)) (goldenwspp.sol#1136)
Apply the check-effects-interactions pattern.

Additional information: link

BaseRfiToken._transfer(address,address,uint256) (goldenwspp.sol#540-593) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(_getLastTransaction(tx.origin) <= block.timestamp,You need to wait TIME_LIMIT_TRANSACTION before perform another transaction.) (goldenwspp.sol#581)
AntiDump._getMultiplier() (goldenwspp.sol#999-1008) uses timestamp for comparisons
Dangerous comparisons:
- block.timestamp < TimeContractStart + 24 * 3600 (goldenwspp.sol#1002)
Ownable.unlock() (safetoken-imports.sol#112-117) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp > _lockTime,The contract is still locked) (safetoken-imports.sol#114)
Avoid relying on block.timestamp.

Additional information: link

Address.isContract(address) (safetoken-imports.sol#36) uses assembly
- INLINE ASM (safetoken-imports.sol#36)
Address._verifyCallResult(bool,bytes,string) (safetoken-imports.sol#66-75) uses assembly
- INLINE ASM (safetoken-imports.sol#69-72)
Do not use evm assembly.

Additional information: link

AntiDump._getMultiplier() (goldenwspp.sol#999-1008) compares to a boolean constant:
-ANTI_DUMP_MULTIPLAYER == true (goldenwspp.sol#1001)
Remove the equality to the boolean constant.

Additional information: link

Address._verifyCallResult(bool,bytes,string) (safetoken-imports.sol#66-75) is never used and should be removed
Address.functionCall(address,bytes) (safetoken-imports.sol#41) is never used and should be removed
Address.functionCall(address,bytes,string) (safetoken-imports.sol#42) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256) (safetoken-imports.sol#43) is never used and should be removed
Address.functionCallWithValue(address,bytes,uint256,string) (safetoken-imports.sol#44-49) is never used and should be removed
Address.functionDelegateCall(address,bytes) (safetoken-imports.sol#58-60) is never used and should be removed
Address.functionDelegateCall(address,bytes,string) (safetoken-imports.sol#61-65) is never used and should be removed
Address.functionStaticCall(address,bytes) (safetoken-imports.sol#50-52) is never used and should be removed
Address.functionStaticCall(address,bytes,string) (safetoken-imports.sol#53-57) is never used and should be removed
Address.isContract(address) (safetoken-imports.sol#36) is never used and should be removed
Address.sendValue(address,uint256) (safetoken-imports.sol#37-40) is never used and should be removed
BaseRfiToken._beforeTokenTransfer(address,address,uint256,bool) (goldenwspp.sol#695) is never used and should be removed
BaseRfiToken._getLastBlock(address) (goldenwspp.sol#747) is never used and should be removed
BaseRfiToken._getLastTransaction(address) (goldenwspp.sol#737) is never used and should be removed
BaseRfiToken._getSumOfFees(address,uint256) (goldenwspp.sol#705) is never used and should be removed
BaseRfiToken._isV2Pair(address) (goldenwspp.sol#710) is never used and should be removed
BaseRfiToken._multiplier() (goldenwspp.sol#757) is never used and should be removed
BaseRfiToken._setLastBlock(address) (goldenwspp.sol#752) is never used and should be removed
BaseRfiToken._setLastTransaction(address) (goldenwspp.sol#742) is never used and should be removed
BaseRfiToken._takeTransactionFees(uint256,uint256,bool,uint256) (goldenwspp.sol#732) is never used and should be removed
Context._msgData() (safetoken-imports.sol#23) is never used and should be removed
Liquifier._approveDelegate(address,address,uint256) (goldenwspp.sol#965) is never used and should be removed
SafeMath.mod(uint256,uint256) (safetoken-imports.sol#30) is never used and should be removed
Tokenomics._addFee(Tokenomics.FeeType,uint256,address) (goldenwspp.sol#215-218) is never used and should be removed
Tokenomics._addFees() (goldenwspp.sol#220-238) is never used and should be removed
Tokenomics.getCollectedFeeTotal(uint256) (goldenwspp.sol#256-259) is never used and should be removed
Remove unused functions.

Additional information: link

Tokenomics.capWhaleWalletBalance (goldenwspp.sol#133) is set pre-construction with a non-constant function or state variable:
- (80 * maxWalletBalance) / 100
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) (safetoken-imports.sol#37-40):
- (success) = recipient.call{value: amount}() (safetoken-imports.sol#38)
Low level call in Address.functionCallWithValue(address,bytes,uint256,string) (safetoken-imports.sol#44-49):
- (success,returndata) = target.call{value: value}(data) (safetoken-imports.sol#47)
Low level call in Address.functionStaticCall(address,bytes,string) (safetoken-imports.sol#53-57):
- (success,returndata) = target.staticcall(data) (safetoken-imports.sol#55)
Low level call in Address.functionDelegateCall(address,bytes,string) (safetoken-imports.sol#61-65):
- (success,returndata) = target.delegatecall(data) (safetoken-imports.sol#63)
Avoid low-level calls. Check the call success. If the call is meant for a contract, check for code existence

Additional information: link

Variable Tokenomics._reflectedSupply (goldenwspp.sol#94) is not in mixedCase
Variable Tokenomics.TIME_LIMIT_TRANSACTION (goldenwspp.sol#96) is not in mixedCase
Variable Tokenomics.TimeContractStart (goldenwspp.sol#97) is not in mixedCase
Variable Tokenomics.ANTI_DUMP_MULTIPLAYER (goldenwspp.sol#98) is not in mixedCase
Constant Tokenomics.maxTransactionAmount (goldenwspp.sol#108) is not in UPPER_CASE_WITH_UNDERSCORES
Constant Tokenomics.numberOfTokensToSwapToLiquidity (goldenwspp.sol#148) is not in UPPER_CASE_WITH_UNDERSCORES
Variable BaseRfiToken._reflectedBalances (goldenwspp.sol#274) is not in mixedCase
Variable BaseRfiToken._balances (goldenwspp.sol#275) is not in mixedCase
Variable BaseRfiToken._allowances (goldenwspp.sol#276) is not in mixedCase
Variable BaseRfiToken._isExcludedFromFee (goldenwspp.sol#278) is not in mixedCase
Variable BaseRfiToken._isExcludedFromRewards (goldenwspp.sol#279) is not in mixedCase
Variable Liquifier._router (goldenwspp.sol#778) is not in mixedCase
Variable Liquifier._pair (goldenwspp.sol#779) is not in mixedCase
Variable AntiBot._lastTransaction (goldenwspp.sol#986) is not in mixedCase
Variable AntiBot._LastBlock (goldenwspp.sol#987) is not in mixedCase
Function IPancakeV2Router.WETH() (safetoken-imports.sol#143) is not in mixedCase
Follow the Solidity naming convention.

Additional information: link

Redundant expression "this (safetoken-imports.sol#23)" inContext (safetoken-imports.sol#21-24)
Remove redundant statements if they congest code but offer no value.

Additional information: link

Variable BaseRfiToken.reflectionFromToken(uint256,bool).rTransferAmount (goldenwspp.sol#474) is too similar to BaseRfiToken._getValues(uint256,uint256).tTransferAmount (goldenwspp.sol#660)
Variable BaseRfiToken._transferTokens(address,address,uint256,bool).rTransferAmount (goldenwspp.sol#630) is too similar to BaseRfiToken._getValues(uint256,uint256).tTransferAmount (goldenwspp.sol#660)
Variable BaseRfiToken._getValues(uint256,uint256).rTransferAmount (goldenwspp.sol#664) is too similar to BaseRfiToken._getValues(uint256,uint256).tTransferAmount (goldenwspp.sol#660)
Variable BaseRfiToken.reflectionFromToken(uint256,bool).rTransferAmount (goldenwspp.sol#474) is too similar to BaseRfiToken._transferTokens(address,address,uint256,bool).tTransferAmount (goldenwspp.sol#630)
Variable BaseRfiToken._getValues(uint256,uint256).rTransferAmount (goldenwspp.sol#664) is too similar to BaseRfiToken._transferTokens(address,address,uint256,bool).tTransferAmount (goldenwspp.sol#630)
Variable BaseRfiToken._transferTokens(address,address,uint256,bool).rTransferAmount (goldenwspp.sol#630) is too similar to BaseRfiToken._transferTokens(address,address,uint256,bool).tTransferAmount (goldenwspp.sol#630)
Variable Liquifier._mainnetRouterV1Address (goldenwspp.sol#770) is too similar to Liquifier._mainnetRouterV2Address (goldenwspp.sol#772)
Prevent variables from having similar names.

Additional information: link

SafeTokenV1Beta.slitherConstructorVariables() (goldenwspp.sol#1132-1138) uses literals with too many digits:
- burnAddress = 0x000000000000000000000000000000000000dEaD (goldenwspp.sol#189)
Use: Ether suffix, Time suffix, or The scientific notation

Additional information: link

Liquifier._mainnetRouterV1Address (goldenwspp.sol#770) should be constant
Liquifier._mainnetRouterV2Address (goldenwspp.sol#772) should be constant
Liquifier._testnetRouterAddress (goldenwspp.sol#776) should be constant
Tokenomics.burnAddress (goldenwspp.sol#189) should be constant
Tokenomics.tipToTheDev (goldenwspp.sol#198) should be constant
Add the constant attributes to state variables that never change.

Additional information: link

increaseAllowance(address,uint256) should be declared external:
- BaseRfiToken.increaseAllowance(address,uint256) (goldenwspp.sol#450-453)
decreaseAllowance(address,uint256) should be declared external:
- BaseRfiToken.decreaseAllowance(address,uint256) (goldenwspp.sol#455-458)
isExcludedFromFee(address) should be declared external:
- BaseRfiToken.isExcludedFromFee(address) (goldenwspp.sol#515)
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (safetoken-imports.sol#94-97)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (safetoken-imports.sol#98-102)
getUnlockTime() should be declared external:
- Ownable.getUnlockTime() (safetoken-imports.sol#103-105)
lock(uint256) should be declared external:
- Ownable.lock(uint256) (safetoken-imports.sol#106-111)
unlock() should be declared external:
- Ownable.unlock() (safetoken-imports.sol#112-117)
manager() should be declared external:
- Manageable.manager() (safetoken-imports.sol#127)
Use the external attribute for functions never called from the contract.

Additional information: link

Contract has 10% buy tax and 9% sell tax.
Taxes are low but contract ownership is not renounced. Token has a high risk of becoming a honeypot.

Average 30d PancakeSwap liquidity is low.

Average 30d number of PancakeSwap swaps is low.

Token is deployed only at one blockchain

Unable to find token on CoinGecko

Additional information: link

Unable to find Telegram link on the website

Unable to find whitepaper link on the website

Unable to find token on CoinHunt

Additional information: link

Token has no active CoinGecko listing / rank

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 CoinMarketCap rank