Defi Yeild - IDO,Token,Vesting

The setMaxInvestment and setMinInvestment functions are incorrectly checking conditions. In setMaxInvestment, it compares the input _investment with _minInvestment which is not the intended comparison. Similarly, in setMinInvestment, it compares _investment with _maxInvestment which is contrary to the expected behavior. These incorrect condition checks can lead to unpredictable behavior and potential vulnerabilities in the investment limits of the IDO project

The _getTokenAmount and _updatePurchasingState functions are not checking the freshness of the price received from the oracle before using it in calculations. This vulnerability can lead to inaccurate conversion rates and investment calculations. If stale or outdated prices are used, it can result in incorrect token amounts being allocated to investors, causing financial losses and potential disputes

The setupBeneficiary and disableBeneficiary functions lack sufficient controls, leading to potential issues in the beneficiary management process.

The LaunchPadErc20Ido contract does not implement a token rescue mechanism, which means there is no way to recover tokens or native tokens (such as ETH) sent to the contract mistakenly or under exceptional circumstances. In the absence of a token rescue feature, any tokens sent to the contract address, either intentionally or accidentally, become irretrievable, leading to potential financial losses for users or the project itself

In the case of stable coins, the hierarchy of authority plays a crucial role and our team recommends that the roles with privileges must be given to the accounts with proven authority and functions like “Minting” and “Burning” must be used cautiously because once these functions are called on the mainnet then could be no minting or burning. Hence, it will be an irreversible change.

Critical functions within the contract lack the emission of events. Events play a vital role in providing transparency, enabling external systems to react to changes, and offering a way to track important contract activities. The absence of events can hinder monitoring and auditing efforts, making it difficult to detect and respond to critical contract actions