Platform: Code4rena
Start Date: 04/11/2021
Pot Size: $50,000 USDC
Total HM: 20
Participants: 28
Period: 7 days
Judge: 0xean
Total Solo HM: 11
Id: 51
League: ETH
Rank: 25/28
Findings: 2
Award: $18.69
🌟 Selected for report: 1
🚀 Solo Findings: 0
🌟 Selected for report: TomFrenchBlockchain
Also found by: PranavG, Reigada, WatchPug, jah, nathaniel, pants, pauliax, pmerkleplant
pmerkleplant
Defining a variable as immutable, if possible, saves gas and increases code clarity.
Following variables should be defined as immutable:
initTime
in vesting/contracts/InvestorDistribution.sol
mainToken
in vesting/contracts/InvestorDistribution.sol
vestLock
in vesting/contracts/InvestorDistribution.sol
mainToken
in tge/contracts/PublicSale.sol
nft
in tge/contracts/PublicSale.sol
vestLock
in tge/contracts/PublicSale.sol
tgeContract
in tge/contracts/PublicSaleBatchWithdraw.sol
#0 - chickenpie347
2022-01-03T23:13:27Z
Duplicate of #1 , #3 , #102
🌟 Selected for report: pmerkleplant
12.1196 USDC - $12.12
pmerkleplant
Removing unused variables saves gas and increases code clarity.
Following variables are unused and can be removed:
Hour
in vesting/contracts/AidropDistribution.sol
Day
in vesting/contracts/AirdropDistribution.sol
Hour
in vesting/contracts/InvestorDistribution.sol
Day
in vesting/contracts/InvestorDistribution.sol
_balance
in tge/contracts/PublicSale.sol
_allowance
in tge/contracts/PublicSale.sol
pmerkleplant
Defining a variable as constant, if possible, saves gas and increases code clarity.
Following variables should be defined as constants:
airdrop_supply
in vesting/contracts/AidropDistribution.sol
investors_supply
in vesting/contracts/InvestorDistribution.sol
#0 - chickenpie347
2022-01-03T23:12:16Z
Duplicate of #88