OpenFundNet Info
OpenFundNet is a decentralized crowdfunding platform built on blockchain. It connects innovators with a global community of funders, while using validators and a milestone-based system to ensure transparency and fairness.
TrustNet Score
The TrustNet Score evaluates crypto projects based on audit results, security, KYC verification, and social media presence. This score offers a quick, transparent view of a project's credibility, helping users make informed decisions in the Web3 space.
Real-Time Threat Detection
Real-time threat detection, powered by Cyvers.io,
is currently not
activated
for this project.
This advanced feature provides continuous monitoring and instant alerts to safeguard your assets from potential security threats. Real-time detection enhances your project's security by proactively identifying and mitigating risks.
For more information, click here.
Security Assessments
Summary and Final Words
No crucial issues found
The contract does not contain issues of high or medium criticality. This means that no known vulnerabilities were found in the source code.
Contract owner cannot mint
It is not possible to mint new tokens.
Contract owner cannot blacklist addresses.
It is not possible to lock user funds by blacklisting addresses.
Contract owner cannot set high fees
The fees, if applicable, can be a maximum of 25% or lower. The contract can therefore not be locked. Please take a look in the comment section for more details.
Contract cannot be locked
Owner cannot lock any user funds.
Token cannot be burned
There is no burning within the contract without any allowances
Ownership is not renounced
The owner retains significant control, which could potentially be used to modify key contract parameters.
Contract is not upgradeable
The contract does not use proxy patterns or other mechanisms to allow future upgrades. Its behavior is locked in its current state.
Scope of Work
This audit encompasses the evaluation of the files listed below, each verified with a SHA-1 Hash. The team referenced above has provided the necessary files for assessment.
The auditing process consists of the following systematic steps:
- Specification Review: Analyze the provided specifications, source code, and instructions to fully understand the smart contract's size, scope, and functionality.
- Manual Code Examination: Conduct a thorough line-by-line review of the source code to identify potential vulnerabilities and areas for improvement.
- Specification Alignment: Ensure that the code accurately implements the provided specifications and intended functionalities.
- Test Coverage Assessment: Evaluate the extent and effectiveness of test cases in covering the codebase, identifying any gaps in testing.
- Symbolic Execution: Analyze the smart contract to determine how various inputs affect execution paths, identifying potential edge cases and vulnerabilities.
- Best Practices Evaluation: Assess the smart contracts against established industry and academic best practices to enhance efficiency, maintainability, and security.
- Actionable Recommendations: Provide detailed, specific, and actionable steps to secure and optimize the smart contracts.
A file with a different Hash has been intentionally or otherwise modified after the security review. A different Hash may indicate a changed condition or potential vulnerability that was not within the scope of this review.
Final Words
The following provides a concise summary of the audit report, accompanied by insightful comments from the auditor. This overview captures the key findings and observations, offering valuable context and clarity.
Ownership Privileges
-
The owner can update token prices without limits (updatePrice). -
The owner can toggle sale on/off at any time (updateSale). -
The owner can change the token being sold mid-sale (updateToken). -
The owner can modify referral percentages without bounds (updateRefferPercentage). -
The owner can withdraw any ERC20 tokens from the contract (withdrawToken). -
The owner can withdraw all raised BNB without timelock (withdrawBNB).
Note - This Audit report consists of a security analysis of the OFNT smart contract. This analysis did not include functional testing (or unit testing) of the contract’s logic. Moreover, we only audited the mentioned contract for the OFNT team. Other contracts associated with the project were not audited by our team. We recommend investors do their own research before investing.
Files and details
Findings and Audit result
medium Issues | 6 findings
Pending
#1 medium Issue
Missing Input Validation in updatePrice Function.
The updatePrice function lacks critical input validation for the usdtPrice parameter, allowing it to be set to zero, unreasonably high values, or drastically different values from the previous price without any restriction. This creates significant economic risks to the token sale, including potential free token distribution, calculation errors, or prohibitively expensive pricing that could disrupt the token sale mechanics. To mitigate this vulnerability, implement bounds checking that ensures the price is always greater than zero, falls below a reasonable maximum threshold, and doesn't change too drastically in a single update (e.g., require(_usdtPrice > 0) and require price changes stay within a reasonable percentage of the previous value).
Pending
#2 medium Issue
Missing 'isContract' check.
The contract lacks a validation check to ensure that specific parameters are contract addresses. Without this check, there is a risk that non-contract addresses (such as externally owned accounts, or EOAs) could be mistakenly set for parameters intended to reference other contracts. This could lead to failures in executing critical interactions within the contract, as EOAs do not support contract-specific functions.
Pending
#3 medium Issue
Missing Input Validation in updateRefferPercentage Function
The updateRefferPercentage function allows the owner to set any arbitrary value for the referral percentage without validation constraints. This is especially concerning as the refferPercentage value directly impacts token distribution calculations in the buy functions where it's used to calculate referral rewards (_referAmount = (_amount.mul(refferPercentage)).div(10000)). Without bounds checks, the owner could potentially set this value excessively high (e.g., >10000), resulting in referral rewards that exceed the principal investment amount, or set it to zero, eliminating referral incentives entirely.
Pending
#4 medium Issue
Incorrect Vesting Calculation in Buy Function
The buy function erroneously stores payment amounts (USDT/WETH) in vestedBalance mapping instead of token amounts, creating a fundamental mismatch between purchased tokens and vested tokens. Since the claim function uses vestedBalance values to calculate token distributions, users will receive significantly fewer tokens than they purchased (e.g., if 100 USDT buys 1000 tokens, users would only receive 100 tokens through vesting). This critical issue undermines the entire vesting mechanism and would result in substantial financial loss for participants.
Pending
#5 medium Issue
No Slippage Protection in buyWithBNB Function
The buyWithBNB function calculates token amounts based on current BNB/token exchange rates at transaction execution time without allowing users to specify a minimum acceptable amount. This creates significant risk during volatile market conditions or high network congestion, where users might receive far fewer tokens than expected when their transaction executes. Without slippage protection, users have no guarantee about the minimum tokens they'll receive for their BNB, potentially resulting in unfavorable executions that still succeed rather than reverting when prices move against them.
Pending
#6 medium Issue
No Verification that Vesting is Complete in claim Function
The claim function only performs basic timing checks (saleEndTime > 0 and block.timestamp >= saleEndTime) to determine if tokens can be claimed, but lacks comprehensive verification that the vesting schedule has been properly initialized and configured. Without validating that all vesting parameters are properly set, there's a risk that claims could begin prematurely or with incorrect parameters. This is particularly concerning given that the vestedPeriod is set to a testing value of 5 minutes and there's no explicit function to properly initialize the vesting schedule when transitioning from the sale to vesting phase.
low Issues | 4 findings
Pending
#1 low Issue
Remove safemath library
The compiler version above 0.8.0 has the ability to control arithmetic overflow/underflow. It is recommended to remove the unwanted code in order to avoid high gas fees.
Pending
#2 low Issue
No Price Change Limits
There are no restrictions on how drastically the price can change between updates. This allows for arbitrary price manipulation that could harm users. The function directly sets usdtPrice = _usdtPrice without checking how much it differs from the previous value.
Pending
#3 low Issue
No Input Validation for Amount
The function accepts any non-zero amount without minimum or maximum limits. No validation like require(_amount > minPurchase && _amount < maxPurchase) is present in the function.
Pending
#4 low Issue
Redundant Token Calculation in Buy Function
The buy function contains a redundant and costly calculation by calling the calculateToken function twice with the same parameters. The function first calculates the token amount with amount = calculateToken(_amount, _pay) and later repeats this exact calculation when storing buyer information: buyerInfo[saleCounter] = buyer(msg.sender, _amount, calculateToken(_amount, _pay), _pay). This redundancy unnecessarily increases gas costs, especially problematic since calculateToken performs expensive operations including external calls to the Uniswap router for price calculations.
informational Issues | 1 findings
Pending
#1 informational Issue
Floating pragma solidity version
Adding the constant version of solidity is recommended, as this prevents the unintentional deployment of a contract with an outdated compiler that contains unresolved bugs.