Valkyrio is a blockchain-based game inspired by a manga. At Valkyrio, players will immerse themselves in the role of a humanity representative by recruiting fighters, forging the weapons, killing monsters, and challenging Gods. At Valkyrio, we believe that having a unique fighter and/or weapon is one of the ways the game can be appealing and engaging, but also rewarding to the players.
Key features:
-Enjoy & Earn: The joy comes by enjoying the gameplay of a RPG game you might be familiar with. You will start by training your recruited fighters daily by combating the monsters, leveling & enhancing your fighter's power.
The game rewards players who join and invest time & funds to enjoy it. Fueled by the dual token system $VALK & $YRIO, its tokenomic stability will be the strong advantage of the game. $VALK will be the “entry” to join our world and $YRIO will be the reward currency for all in-game activities.
-Marketplace: Fighters & the assets (weapons, outfit, valkyries blessing,…can be traded on the marketplace)
-Game events:
GODs challenge: Every week there will be a boss event. All players can participate and win the treasury in the event.
PvP leaderboard: Every week there will be a PvP leaderboard ranking competition. All players who have PvP matches during the time of event will automatically signup & compete for the event’s reward.
Games in-game: The big features of the game will come in the next phase of the project. Players will have the ability of owning land. By becoming a landlord, players will unlock a series of minigames inside their land (planting, fishing, mining,..). All those minigame activities will be rewarded. The land also gives fighter bonus powers for combating and can be traded on the marketplace.
Unable to verify that contract auditor is trusted: Certik, Quantstamp, Hacken, Solidity, Paladinsec, Openzeppelin, Verichains
ValkToken._transfer(address,address,uint256).transferFeeRate (#1032) 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
Reentrancy in ValkToken._transfer(address,address,uint256) (#1018-1047):
External calls:
- BP.protect(sender,recipient,amount) (#1029)
State variables written after the call(s):
- super._transfer(sender,address(this),_fee) (#1042)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#539)
- _balances[recipient] = _balances[recipient].add(amount) (#540)
- super._transfer(sender,recipient,amount) (#1046)
- _balances[sender] = _balances[sender].sub(amount,ERC20: transfer amount exceeds balance) (#539)
- _balances[recipient] = _balances[recipient].add(amount) (#540)
Reentrancy in ValkToken.constructor(address) (#923-935):
External calls:
- uniswapV2Pair = IUniswapV2Factory(_uniswapV2Router.factory()).createPair(address(this),_uniswapV2Router.WETH()) (#930-931)
State variables written after the call(s):
- blacklistTime = block.timestamp + 259200 (#934)
- uniswapV2Router = _uniswapV2Router (#933)
Apply the check-effects-interactions pattern.
Additional information: link
Reentrancy in ValkToken._transfer(address,address,uint256) (#1018-1047):
External calls:
- BP.protect(sender,recipient,amount) (#1029)
Event emitted after the call(s):
- Transfer(sender,recipient,amount) (#541)
- super._transfer(sender,recipient,amount) (#1046)
- Transfer(sender,recipient,amount) (#541)
- super._transfer(sender,address(this),_fee) (#1042)
Apply the check-effects-interactions pattern.
Additional information: link
ValkToken.setBlackLists(address[]) (#958-968) uses timestamp for comparisons
Dangerous comparisons:
- require(bool,string)(block.timestamp <= blacklistTime,BLACKLISTTIME: INVALID) (#959)
Avoid relying on block.timestamp.
Additional information: link
ValkToken.setBotProtectionDisableForever() (#987-990) compares to a boolean constant:
-require(bool)(BPDisabledForever == false) (#988)
Remove the equality to the boolean constant.
Additional information: link
Different versions of Solidity is used:
- Version used: ['>=0.5.0', '>=0.6.0<0.8.0', '>=0.6.2', '>=0.6.5', '^0.6.5']
- >=0.6.0<0.8.0 (#5)
- >=0.6.0<0.8.0 (#32)
- >=0.6.0<0.8.0 (#112)
- >=0.6.0<0.8.0 (#329)
- >=0.6.0<0.8.0 (#637)
- >=0.6.2 (#705)
- >=0.6.2 (#803)
- >=0.5.0 (#849)
- >=0.6.5 (#870)
- ^0.6.5 (#883)
Use one Solidity version.
Additional information: link
Context._msgData() (#22-25) is never used and should be removed
ERC20._setupDecimals(uint8) (#612-614) is never used and should be removed
SafeMath.div(uint256,uint256,string) (#299-302) is never used and should be removed
SafeMath.mod(uint256,uint256) (#261-264) is never used and should be removed
SafeMath.mod(uint256,uint256,string) (#319-322) is never used and should be removed
SafeMath.tryAdd(uint256,uint256) (#133-137) is never used and should be removed
SafeMath.tryDiv(uint256,uint256) (#169-172) is never used and should be removed
SafeMath.tryMod(uint256,uint256) (#179-182) is never used and should be removed
SafeMath.tryMul(uint256,uint256) (#154-162) is never used and should be removed
SafeMath.trySub(uint256,uint256) (#144-147) is never used and should be removed
Remove unused functions.
Additional information: link
Pragma version>=0.6.0<0.8.0 (#5) is too complex
Pragma version>=0.6.0<0.8.0 (#32) is too complex
Pragma version>=0.6.0<0.8.0 (#112) is too complex
Pragma version>=0.6.0<0.8.0 (#329) is too complex
Pragma version>=0.6.0<0.8.0 (#637) is too complex
Pragma version>=0.6.2 (#705) allows old versions
Pragma version>=0.6.2 (#803) allows old versions
Pragma version>=0.5.0 (#849) allows old versions
Pragma version>=0.6.5 (#870) allows old versions
Pragma version^0.6.5 (#883) allows old versions
solc-0.6.5 is not recommended for deployment
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
Function IUniswapV2Router01.WETH() (#709) is not in mixedCase
Parameter ValkToken.setAddressForBosses(address)._addressForBosses (#937) is not in mixedCase
Parameter ValkToken.setWhiteLists(address)._target (#944) is not in mixedCase
Parameter ValkToken.removeWhiteLists(address)._target (#951) is not in mixedCase
Parameter ValkToken.setBPAddrss(address)._bp (#977) is not in mixedCase
Parameter ValkToken.setBpEnabled(bool)._enabled (#983) is not in mixedCase
Variable ValkToken.BP (#905) is not in mixedCase
Variable ValkToken.BPDisabledForever (#907) is not in mixedCase
Follow the Solidity naming convention.
Additional information: link
Redundant expression "this (#23)" inContext (#17-26)
Remove redundant statements if they congest code but offer no value.
Additional information: link
Variable IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountADesired (#714) is too similar to IUniswapV2Router01.addLiquidity(address,address,uint256,uint256,uint256,uint256,address,uint256).amountBDesired (#715)
Prevent variables from having similar names.
Additional information: link
ValkToken.buyFeeRate (#914) should be constant
ValkToken.maxSupply (#909) should be constant
ValkToken.sellFeeRate (#913) should be constant
Add the constant attributes to state variables that never change.
Additional information: link
name() should be declared external:
- ERC20.name() (#389-391)
symbol() should be declared external:
- ERC20.symbol() (#397-399)
decimals() should be declared external:
- ERC20.decimals() (#414-416)
totalSupply() should be declared external:
- ERC20.totalSupply() (#421-423)
transfer(address,uint256) should be declared external:
- ERC20.transfer(address,uint256) (#440-443)
allowance(address,address) should be declared external:
- ERC20.allowance(address,address) (#448-450)
approve(address,uint256) should be declared external:
- ERC20.approve(address,uint256) (#459-462)
transferFrom(address,address,uint256) should be declared external:
- ERC20.transferFrom(address,address,uint256) (#477-481)
increaseAllowance(address,uint256) should be declared external:
- ERC20.increaseAllowance(address,uint256) (#495-498)
decreaseAllowance(address,uint256) should be declared external:
- ERC20.decreaseAllowance(address,uint256) (#514-517)
renounceOwnership() should be declared external:
- Ownable.renounceOwnership() (#687-690)
transferOwnership(address) should be declared external:
- Ownable.transferOwnership(address) (#696-700)
setBlackLists(address[]) should be declared external:
- ValkToken.setBlackLists(address[]) (#958-968)
removeBlackLists(address[]) should be declared external:
- ValkToken.removeBlackLists(address[]) (#970-975)
burn(uint256) should be declared external:
- ValkToken.burn(uint256) (#992-994)
sweepTokenForBosses() should be declared external:
- ValkToken.sweepTokenForBosses() (#996-999)
Use the external attribute for functions never called from the contract.
Additional information: link
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
Young tokens have high risks of price dump / death
Token has relatively low CoinGecko rank
Token has relatively low CoinMarketCap rank
Last post in Twitter was more than 30 days ago
Unable to find Blog account (Reddit or Medium)
Unable to find Youtube account
Unable to find Discord account